[PATCH] comctl32: Fixed strncpy (Coverity)

[Michael Stefaniuc]

Hello Marcus,

On 06/28/2011 09:15 AM, Marcus Meissner wrote:
> This is a 2 CHAR buffer, and we copy in 1 CHAR and \0. So just do that
> via strcpy() and dont confuse the static checker.
> 
> CID 1194
> 
> Ciao, Marcus
> ---
>  dlls/comctl32/tests/treeview.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/dlls/comctl32/tests/treeview.c b/dlls/comctl32/tests/treeview.c
> index 324473d..faa9553 100644
> --- a/dlls/comctl32/tests/treeview.c
> +++ b/dlls/comctl32/tests/treeview.c
> @@ -1225,7 +1225,7 @@ static void test_itemedit(void)
>      r = TreeView_SelectItem(hTree, NULL);
>      expect(TRUE, r);
>      /* alter text */
> -    strncpy(buff, "x", sizeof(buff)/sizeof(CHAR));
> +    strcpy(buff, "x");
>      r = SendMessage(edit, WM_SETTEXT, 0, (LPARAM)buff);
What about
       r = SendMessage(edit, WM_SETTEXT, 0, (LPARAM)"x");
No strcpy involved.

>      expect(TRUE, r);
>      r = SendMessage(hTree, WM_COMMAND, MAKEWPARAM(0, EN_KILLFOCUS), (LPARAM)edit);

bye
	michael