[PATCH rebase] peflags: Fix ULONG range checks
Christian Franke
Christian.Franke@t-online.de
Tue Aug 8 08:40:54 GMT 2023
Corinna Vinschen wrote:
> On Aug 8 10:06, Christian Franke via Cygwin-apps wrote:
>> Corinna Vinschen via Cygwin-apps wrote:
>>> Hi Christian,
>>>
>>> On Aug 7 16:07, Christian Franke via Cygwin-apps wrote:
>>>> Minor issue found during tests of the upcoming 'peflags --timestamp' patch.
>>>>
>>>> --
>>>> Regards,
>>>> Christian
>>>>
>>>> ...
>>>> diff --git a/peflags.c b/peflags.c
>>>> index 93eaa0b..d98b121 100644
>>>> --- a/peflags.c
>>>> +++ b/peflags.c
>>>> @@ -30,7 +30,6 @@
>>>> #include <unistd.h>
>>>> #include <getopt.h>
>>>> #include <errno.h>
>>>> -#include <limits.h>
>>>> #if defined (__CYGWIN__) || defined (__MSYS__)
>>>> #include <sys/mman.h>
>>>> #endif
>>>> @@ -598,7 +597,7 @@ handle_num_option (const char *option_name,
>>>> || sizeof_vals[option_index].value > 0x0000ffffffffffffULL
>>>> /* Just a ULONG value */
>>>> || (sizeof_vals[option_index].is_ulong
>>>> - && sizeof_vals[option_index].value > ULONG_MAX))
>>>> + && sizeof_vals[option_index].value > 0x00000000ffffffffULL))
>>> What about using MAXDWORD or MAXULONG32 instead?
>> Of course :-)
>>
>> Christian
>>
> Pushed. I've started deploying a new release.
I'm currently working on 'rebase -c, --checksum' and found one minor
issue: pechecksum.* are missing in SRC_DISTFILES.
More information about the Cygwin-apps
mailing list