Re: [PATCH 1/3] cabinet/tests: Alter string declaration to include null terminator.

[Dmitry Timoshkov]

Isira Seneviratne <isirasen96(a)gmail.com> wrote:
> + * Copyright (C) 2019 Isira Seneviratne

Does this really deserve adding the copyright notice?
> *
>   * This library is free software; you can redistribute it and/or
>   * modify it under the terms of the GNU Lesser General Public
> @@ -776,7 +777,7 @@ static UINT CDECL fdi_mem_read(INT_PTR hf, void *pv, UINT cb)
>
> static UINT CDECL fdi_mem_write(INT_PTR hf, void *pv, UINT cb)
>  {
> -    static const char expected[12] = "Hello World!";
> +    static const char expected[12] = { 'H','e','l','l','o','W','o','r','l','d','!','\0' };

It was deliberate choice, nohing depends on it being 0 terminated.

-- 
Dmitry.