[perl #121712] [PATCH] Coverity: regcomp.c etc.

On Tue Apr 22 17:33:07 2014, jhi wrote:
> Needed compile-time limits for the PL_reg_intflags_name so that the
> bit loop doesn't waltz off past the array.  Could not use C_ARRAY_LENGTH
> because the size of name array is not visible during compile time
> (only const char*[] is), so modified regcomp.pl to generate the size,
> made it visible only for the re extension.  Did extflags analogously
> even though its size currently exactly 32 already.  The sizeof(flags)*8
> is extra paranoia for ILP64.

Wouldn't the paranoia in:

+    for (bit=0;
+         bit<REG_EXTFLAGS_NAME_SIZE && bit<sizeof(flags)*8;
+         bit++) {

be better handled with an assert?  Even a compile-time assert if we had the mechanism for it.

Tony

---
via perlbug:  queue: perl5 status: new
https://rt.perl.org/Ticket/Display.html?id=121712

• Re: [perl #121712] [PATCH] Coverity: regcomp.c etc. by Jarkko Hietaniemi
• [perl #121712] [PATCH] Coverity: regcomp.c etc. by bulk88 via RT
• Re: [perl #121712] [PATCH] Coverity: regcomp.c etc. by Jarkko Hietaniemi
• [perl #121712] [PATCH] Coverity: regcomp.c etc. by Tony Cook via RT