gh-127740: For odd-length input to bytes.fromhex(...) change the er 8000 ror message to ValueError: fromhex() arg must be of even length #127756
Conversation
…e error message to ValueError: fromhex() arg must be of even length
|
@hauntsaninja Please review again? |
|
Yes, I agree with @picnixz and @hauntsaninja, the new if is done for every character pair now, which makes things less efficient. I suggest moving the new check into the condition just below. |
|
No, that's not what I meant, sorry for being unclear. many invalid byte values are mapped to 37, not just null. So instead, you need to check for your original condition |
|
@cfbolz what do you think now ? |
|
@srinivasreddy code looks good now, imo. please add a news entry though. |
| @@ -459,6 +459,12 @@ def test_fromhex(self): | |||
| self.assertRaises(ValueError, self.type2test.fromhex, '\x00') | |||
| self.assertRaises(ValueError, self.type2test.fromhex, '12 \x00 34') | |||
|
|
|||
| # For odd number of character(s) | |||
| for value in ("a", "aaa", "deadbee"): | |||
There was a problem hiding this comment.
nit: would be nice to have a test case where the string is of even length, e.g. " aaa" and test cases where we check the error message for " aa a " / " aa a a "
There was a problem hiding this comment.
sorry for going ahead and merging @hauntsaninja. I should have waited for more input from you.
There was a problem hiding this comment.
oh all good, was just a nit and commits are cheap :-) thanks for reviewing!
|
That was fast. Thanks. |
…e error message to ValueError: fromhex() arg must be of even length (python#127756)
bytes.fromhexis cryptic #127740