[Bug 18151] New: strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Summary: strtol - errno is not set
           Product: Wine
           Version: 1.1.16
          Platform: PC
        OS/Version: other
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

The output of following program :
=================================
#include <stdlib.h>
#include <stdio.h>
#include <errno.h>

int main () {
  long l = -1;
  char *s, *e;

/*s = "18446744073709551615LL";*/
  s = "9223372036854775807L";
  l = strtol(s, &e, 0);
  fprintf(stderr, "l=%ld errno=%d\n", l, errno);

return (0);
}
=================================
cross-compiled with mingw and run under wine is
l=2147483647 errno=0

Same program run on w2k output:
l=2147483647 errno=34

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, source

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |[e-mail]
          Component|-unknown                    |msvcrt
     Ever Confirmed|0                           |1

--- Comment #1 from Louis Lenders   2009-08-23 18:17:25 ---
confirming. could you add the test (to msvcrt/tests/string.c)

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Eric Pouech  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Eric Pouech   2009-08-29 15:36:51 ---
patch sent to wine-patch (for both strtol and strtoul), test case derivated
from the code in this PR included as well
A+

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Wilfried Pasquazzo  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
                   |                            |m

--- Comment #3 from Wilfried Pasquazzo   2009-09-23 14:47:50 ---
Bug is fixed, can be closed now.

Patch got commited a while ago and the test program now gives the correct
errorcode in Wine:

me@home ~ $ wine test.exe
l=2147483647 errno=34

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Nikolay Sivov   2009-09-23 14:53:12 ---
Fixed by commit:

http://source.winehq.org/git/wine.git/?a=commit;h=9eca2f81dc92f6e28ebe757262...

[Bug 18151] strtol - errno is not set

http://bugs.winehq.org/show_bug.cgi?id=18151

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Alexandre Julliard   2009-09-25 12:25:27 ---
Closing bugs fixed in 1.1.30.