regcomp.c: Fix potential scalar leak
authorKarl Williamson <public@khwilliamson.com>
Fri, 26 Jul 2013 20:26:27 +0000 (14:26 -0600)
committerKarl Williamson <public@khwilliamson.com>
Tue, 30 Jul 2013 18:05:46 +0000 (12:05 -0600)
commit1fa1147c2c277eb9a22c643f21b08da9bc15510e
tree4ca5b4e91857c87244c0d3e6041517ef7bc15d46
parent15c0b4aa89b8fe74b8f5e4712967bbb49292b95e
regcomp.c: Fix potential scalar leak

The lines in this code were reversed.  We need to check something before
overwriting it, rather than the other way around.  The result would be
that under certain circumstances a SV would not get freed.  Those
circumstances are very limited: the first of the three parameters to
this function is not empty, but the 2nd is, and the output (3rd
parameter) is to overwrite the 2nd.  I found this bug by code reading; I
have searched the code space and there are no current calls to it that
have this parameter configuration, therefore there is no test that can
be added to trigger it.
regcomp.c