![](https://secure.gravatar.com/avatar/39b623a1559cf9c69ac3d9d4fb44e7fe.jpg?s=120&d=mm&r=g)
15 Aug
2021
15 Aug
'21
12:11 a.m.
Hi, Sachin! On Aug 10, Sachin Setiya wrote:
What does the comment mean? It's the code from the last millenium, does it even apply now? For example, there can be no \0 bytes in the string, because of
if (!real_binary_mode && strlen(line) != line_length)
There can be \0 bytes in the string in the case of binary mode.
Sure, but in that case we don't want to break out of the loop. I mean, I can see no case when we *do* want to break out of the loop on \0 here. That's why I wonder:
in other words, shouldn't we just remove the if? I tried, at least the main suite didn't break.
Regards, Sergei VP of MariaDB Server Engineering and security@mariadb.org