From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2155) id D556C3858D33; Tue, 8 Aug 2023 08:31:11 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D556C3858D33 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1691483471; bh=O2CLy+oLTrWhE/B61gu/AnQ6Frol3HMWla/bPtvjzng=; h=To:Subject:Date:From:From; b=OsL6cWilt2Z6E1NWjjSK3ayRXtgUc9ZskbNCVR6PcCQZGwWUT5to+CGcqNPQqw8Pl R6Vb8634g0NC3BdKbNq037EMkMJboQPLgp87JSKkVKUGDxTR8LjnqMdAmeqkiA6kGp 75LEYhnK4DR2zYuRHv130FiaGCHHZiy7JQc5pdqE= To: cygwin-apps-cvs@sourceware.org Subject: [rebase - The rebase tool, core of the automatic rebase facility during postinstall] branch master, updated. fa4229d7099fbf14e83125fed142c03638574d50 X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: bb79f3cdb48f90948ccc41621f77f32ab59495cb X-Git-Newrev: fa4229d7099fbf14e83125fed142c03638574d50 Message-Id: <20230808083111.D556C3858D33@sourceware.org> Date: Tue, 8 Aug 2023 08:31:11 +0000 (GMT) From: Corinna Vinschen List-Id: https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/rebase.git;h=fa4229d7099fbf14e83125fed142c03638574d50 commit fa4229d7099fbf14e83125fed142c03638574d50 Author: Christian Franke Date: Tue Aug 8 09:58:39 2023 +0200 peflags: Fix ULONG range checks Don't use ULONG_MAX from because ULONG is not necessarily 'unsigned long'. Use MAXULONG32 instead. Signed-off-by: Christian Franke Diff: --- peflags.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/peflags.c b/peflags.c index f215704dff31..1a61da714d73 100644 --- a/peflags.c +++ b/peflags.c @@ -30,7 +30,6 @@ #include #include #include -#include #if defined (__CYGWIN__) || defined (__MSYS__) #include #endif @@ -696,7 +695,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 > MAXULONG32)) { fprintf (stderr, "Invalid argument for %s: %s\n", option_name, option_arg); @@ -1092,7 +1091,7 @@ get_and_set_size (const pe_file *pep, sizeof_values_t *val) } else if (val->handle == DO_WRITE) { - if ((!pep->is_64bit || val->is_ulong) && val->value >= ULONG_MAX) + if ((!pep->is_64bit || val->is_ulong) && val->value > MAXULONG32) { fprintf (stderr, "%s: Skip writing %s, value too big\n", pep->pathname, val->name);