From: Pádraig Brady Date: Wed, 18 Feb 2015 22:37:31 +0000 (+0000) Subject: getopt: give accurate ambiguity diagnostic on mem exhaustion X-Git-Tag: v1.0~7137 X-Git-Url: https://gitweb.git.savannah.gnu.org/gitweb/?a=commitdiff_plain;h=4ad02492559a3b95e0ab86d79dd0e0e611c0ac1d;p=gnulib.git getopt: give accurate ambiguity diagnostic on mem exhaustion * lib/getopt.c (_getopt_internal_r): The previous commit broke out the loop too early, which could give a false indication of ambiguous options under memory exhaustion. --- diff --git a/lib/getopt.c b/lib/getopt.c index 6474ba74c9..212cbf7341 100644 --- a/lib/getopt.c +++ b/lib/getopt.c @@ -527,6 +527,8 @@ _getopt_internal_r (int argc, char **argv, const char *optstring, pfound = p; indfound = option_index; } + else if (ambig) + ; /* Taking simpler path to handling ambiguities. */ else if (long_only || pfound->has_arg != p->has_arg || pfound->flag != p->flag @@ -539,10 +541,9 @@ _getopt_internal_r (int argc, char **argv, const char *optstring, struct option_list *newp = malloc (sizeof (*newp)); if (newp == NULL) { - ambig = 1; /* Use simpler fallback message. */ free_option_list (ambig_list); ambig_list = NULL; - break; + ambig = 1; /* Use simpler fallback message. */ } else #endif