Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#5931 closed enhancement (fixed)

Replace strdup() to CPLStrdup()

Reported by: yzhong Owned by: warmerdam
Priority: normal Milestone: 2.0.0
Component: default Version: svn-trunk
Severity: normal Keywords: strdup
Cc:

Description

The strdup() API is deprecated under Windows. I ma having memory heap corruption isssue under Windows 8. A while a go, I file a ticket about this. But I missed one strdup() in gdal/port/cpl_vsisimple.cpp. Apparently, it is also problematic. I have replace it with CPLStrdup() and everything works fine afterwords.

Attachments (1)

strdup.diff (371 bytes) - added by yzhong 7 years ago.

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by yzhong

Attachment: strdup.diff added

comment:1 Changed 7 years ago by yzhong

Resolution: fixed
Status: newclosed

Slightly differently implemented in r28971

comment:2 Changed 6 years ago by Even Rouault

Milestone: 2.02.0.0

Milestone renamed

Note: See TracTickets for help on using tickets.