Ticket #1547 (new defect)

Opened 1 year ago

Last modified 1 year ago

Patch for mingw bug in strdup

Reported by: mloskot Assigned to: warmerdam
Priority: normal Milestone:
Component: default Version: 1.4.0
Severity: normal Keywords:
Cc: rob.mcmullen@gmail.com, dmorissette

Description

NOTE: This patch was posted on the gdal-dev by Rob McMullen?

Using MinGW, there's a really strange bug that I can't yet isolate completely, but occurs within msvcrt's strdup implementation. Enclosed is a patch to duplicate the functionality of strdup but using VSIMalloc, avoiding the problematic strdup.

If you didn't want to use this to replace strdup entirely, you could wrap the new code with ifdefs:

#if defined(WIN32) && defined(GCC)

which should isolate this to MinGW only.

Rob

Attachments

strdup-mingw-bug.patch (0.6 kB) - added by mloskot on 04/01/07 12:53:25.
Patch fixing reported issue

Change History

04/01/07 12:53:25 changed by mloskot

  • attachment strdup-mingw-bug.patch added.

Patch fixing reported issue

04/01/07 13:37:35 changed by warmerdam

While the suggested patch looks relatively harmless, I find it hard to imagine that strdup() is really broken, or that the malloc() used by VSIMalloc() is going to give a different kind of memory (ie. different heap) than what is returned by strdup(). So I'm not prepared to take any action on this without better information to reproduce the problem - followed by an analysis of the root of the problem.

I'll leave this open for a bit in case you want to follow up with more info. Otherwise I'll close it in a bit.

04/01/07 16:12:00 changed by robm

I understand your skepticism, but this is a real problem, and the patch fixed it. Obviously MinGW isn't a commonly used platform, so this isn't critical for anyone but me, probably. I was having a very difficult time debugging it at first, because I was using it through ngpython's swig interface and couldn't debug it with either gdb or ms tools. 150 debug print statements later, I isolated the strdup problem. Now that I know that it's not in the python part, I can write a test using C++ that doesn't mix code with the msvcrt code in Python. That I should be able to debug using gdb.

04/01/07 16:27:40 changed by mloskot

Rob, I have taken some time to research this problem with Google and I'm inclined to state that this is but is related to MinGW, but to GDAL.

As I found, there are plenty of crash reports when using strdup/free with MinGW. Obviously, there are incompatibilities between C/C++ libraries.

I'm not going to judge if it's worth to apply your patch to GDAL or not, but I'd suggest you to write about this to MinGW list as well, may be there are some patches or resolutions well known.

04/02/07 10:35:10 changed by dmorissette

  • cc changed from rob.mcmullen@gmail.com to rob.mcmullen@gmail.com, dmorissette.