bugGluster - Bugs: bug #21860, use FILENAME_MAX instead of...

 
 

bug #21860: use FILENAME_MAX instead of hard-coded 4096

Submitted by:  LI Daobing <lidaobing>
Submitted on:  Fri 21 Dec 2007 02:19:04 AM UTC  
 
Category: GlusterFSSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Porting Issue
Status: FixedPrivacy: Public
Assigned to: NoneOriginator Name: Amar
Open/Closed: ClosedRelease: 
Operating System: NoneReproducibility: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 29 Aug 2008 08:08:55 PM UTC, comment #1:

There are problems with using PATH_MAX as it different on different platforms, and hence it will cause buffer overflow.. but current 1.4 branch doesn't use any hardcoded values, instead uses macros everywhere.

Closing the bug.

Amar Tumballi <amarts>
Project Member
Fri 21 Dec 2007 02:19:04 AM UTC, original submission:

Hello,

there are several places where you use 4096 to denote the longest filename or longest path. How about replace it with the portable macro: FILENAME_MAX and PATH_MAX?

affected file:
fuse-bridge.c
unify-self-heal.c
unify.h
trash.c
server-protocol.c
posix.c

Thanks

LI Daobing <lidaobing>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by amarts (Posted a comment)
  • -unavailable- added by lidaobing (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 29 Aug 2008 08:09:13 PM UTCamartsOpen/ClosedOpen=>Closed
    Fri 29 Aug 2008 08:08:54 PM UTCamartsStatusNone=>Fixed
      Originator Name=>Amar

    Back to the top


    Powered by Savane 3.1-cleanup1