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

Dmitry Timoshkov dmitry at baikal.ru
Sun Aug 18 09:29:24 CDT 2019


Isira Seneviratne <isirasen96 at 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.



More information about the wine-devel mailing list