<div dir="ltr">Hi guys,<div><br></div><div>I checked the fix suggested in the bug description and it is correct.</div><div><br></div><div>Changing</div><div><div><pre style="margin-right:1em;margin-left:1.6em;padding:2px 2px 2px 0px;background-color:rgb(250,250,250);border:1px solid rgb(218,218,218);width:auto;color:rgb(72,72,72);font-size:12px">
ret = regcomp(&regex, "^\\(0x\\)\\{0,1\\}[0-9][0-9]*\\([kKMG]\\{0,1\\}\\)$", 0);</pre></div></div><div>to </div><div><br></div><div><span style="color:rgb(72,72,72);font-family:monospace;font-size:12px;white-space:pre;background-color:rgb(250,250,250)">ret = regcomp(&regex, "^(((0x|0X)[0-9A-Fa-f]+)|(0[0-7]*)|([1-9][0-9]*))[kKMG]?$", REG_EXTENDED);</span><br>
</div><div><span style="color:rgb(72,72,72);font-family:monospace;font-size:12px;white-space:pre;background-color:rgb(250,250,250)"><br></span></div><div><span style="color:rgb(72,72,72);font-family:monospace;font-size:12px;white-space:pre;background-color:rgb(250,250,250)">will take care of the </span>following three problems that exist in the first regex</div>
<div><br></div><div><ul style="margin-bottom:1em;color:rgb(72,72,72);font-family:Verdana,sans-serif;font-size:12px;background-color:rgb(255,255,221)"><li>It accepts a leading 0 (without a following x or X) to represent an octal value, but simultaneously accepts the digits 8 and 9 in said octal value (this yields –1).</li>
<li>It does not support the leading 0X hexadecimal indicator but does support a leading 0x (see following).</li><li>It accepts a leading 0x to represent an hexadecimal value but rejects the a..f and A..F digits in said value.</li>
</ul></div><div><br></div><div>Please let me know your views, and then I can submit a patch for this.</div><div><br></div><div>Thanks and regards,</div><div>Sandeep.</div><div><br></div></div>