Ticket #1681 (new defect)

Opened 1 year ago

Last modified 7 months ago

Add cast to (unsigned char) in all isspace() calls

Reported by: dmorissette Assigned to: warmerdam
Priority: normal Milestone: 1.6.0
Component: default Version: unspecified
Severity: normal Keywords:
Cc: mloskot

Description

Ticket #1678 pointed an issue with isspace() that requires casting of the char passed as argument to (unsigned char). The original issue has been fixed in the MITAB driver in ticket 1678, but there are 28 other places using isspace() in the current trunk version of GDAL that may need to be fixed as well (see attached list).

The issue is explained here: http://www.greenend.org.uk/rjk/2001/02/cfu.html#ctypechar

This ticket is to track this.

Attachments

isspace.txt (2.0 kB) - added by dmorissette on 06/21/07 15:11:49.
List of calls to isspace() in GDAL trunk

Change History

06/21/07 15:11:49 changed by dmorissette

  • attachment isspace.txt added.

List of calls to isspace() in GDAL trunk

06/21/07 15:39:30 changed by mloskot

Here is longer discussion about this issue:

Assertion in .tab driver

12/11/07 10:08:36 changed by warmerdam

  • milestone changed from 1.5.0 to 1.6.0.

12/20/07 09:29:49 changed by mloskot

  • cc set to mloskot.