Opened 14 years ago

Closed 14 years ago

#5698 closed bug (fixed)

[patch] ReiserFS label support

Reported by: wtachi Owned by: bonefish
Priority: normal Milestone: R1
Component: File Systems/ReiserFS Version: R1/Development
Keywords: gsoc2010 Cc:
Blocked By: Blocking:
Platform: All

Description

This patch adds support for labels (volume names) on ReiserFS filesystems.

Attachments (1)

reiserfs-label.patch​ (2.9 KB ) - added by wtachi 14 years ago.
Whoops, I missed that last strlen(). Patch updated to not use strlcpy().

Download all attachments as: .zip

Change History (7)

comment:1 by korli, 14 years ago

GetLabel could return something like a status_t instead of an int.

comment:2 by wtachi, 14 years ago

Patch changed to use B_ENTRY_NOT_FOUND or B_OK. (Is B_ENTRY_NOT_FOUND okay? It's the best thing I could find...)

comment:3 by mmadia, 14 years ago

Component: File Systems → File Systems/ReiserFS

Updating ticket for new component File Systems/ReiserFS.

comment:4 by wtachi, 14 years ago

Keywords: gsoc2010 added

comment:5 by bonefish, 14 years ago

Owner: changed from nobody to bonefish
Status: new → assigned

Thanks for the patch! It looks good with the exception of the strlcpy() use in SuperBlock::GetLabel(). The second parameter must be a null-terminated string, which is not guaranteed to be the case here.

by wtachi, 14 years ago

Attachment: reiserfs-label.patch​ added

Whoops, I missed that last strlen(). Patch updated to not use strlcpy().

comment:6 by bonefish, 14 years ago

Resolution: → fixed
Status: assigned → closed

Thanks! Committed in hrev36132.

Note: See TracTickets for help on using tickets.