bugGnome Chemistry Utils - Bugs: bug #22023, strndup() missing on BSD

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #22023: strndup() missing on BSD

Submitted by:  Daniel Leidert <dleidert>
Submitted on:  Tue 15 Jan 2008 01:56:33 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Jean Bréfort <jean_brefort>
Open/Closed: Closed

Mon 14 Jul 2008 10:11:41 AM UTC, comment #3:

There has been releases since this one was fixed, so closing

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Sat 08 Mar 2008 09:56:42 AM UTC, comment #2:

Fixed in both branches

(file #15209)

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Tue 15 Jan 2008 06:17:27 AM UTC, comment #1:

Thanks for pointing to this issue. My code is just buggy (memory leak). We should use strncpy instead of strndup as the memory is already allocated.

Jean Bréfort <jean_brefort>
Project AdministratorIn charge of this item.
Tue 15 Jan 2008 01:56:33 AM UTC, original submission:

Because strndup is missing on BSDs, these guys have to patch the source a bit. Check

http://www.freebsd.org/cgi/cvsweb.cgi/ports/science/gchemutils/files/patch-gcu-formula.cc?rev=1.2

Can we maybe add a fix to the upstream code? Is there a macro we can use? Could we replace it with g_strndup?

Daniel Leidert <dleidert>
Project Member

 

Attached Files
file #15291:  22023.patch added by jean_brefort (716B - application/octet-stream - really fixes the issue)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jean_brefort (Posted a comment)
  • -unavailable- added by dleidert (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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 14 Jul 2008 10:11:41 AM UTCjean_brefortOpen/ClosedOpen=>Closed
    Tue 18 Mar 2008 01:31:37 PM UTCjean_brefortAttached File-=>Added 22023.patch, #15291
    Tue 18 Mar 2008 01:31:10 PM UTCjean_brefortAttached File#15209=>Removed
    Sat 08 Mar 2008 09:56:42 AM UTCjean_brefortAttached File-=>Added 22023.patch, #15209
      StatusNone=>Fixed
      Assigned toNone=>jean_brefort

    Back to the top


    Powered by Savane 3.1-cleanup1