Re: [PATCH 1/2] setupapi/tests: Add a test for SetupGetStringField with strings with embedded \0.

[Dmitry Timoshkov]

Alexandre Julliard  wrote:
> Dmitry Timoshkov  writes:
> > Signed-off-by: Dmitry Timoshkov
> >
> > dlls/setupapi/tests/parser.c | 69 +++++++++++++++++++++++++++++++-----
> >  1 file changed, 60 insertions(+), 9 deletions(-)
>
> Please update the existing parser tests instead.

Unfortunately existing tests depend on test data being NUL terminated,
and even the test that presumably is supposed to test embedded NUL
(test_key_names() with "abcd=ef\x0gh") fails to do so because of strcpy/
strcat used to create test file contents. That's why I decided to go
a simpler way and just create new test from scratch.
> And if null chars need
> special handling this would need to be done as part of the parser state
> machine.

Thanks.