user32/edit: Check for Null before strlen(try2)

André Hentschel nerv at dawncrow.de
Sat Jun 6 07:21:37 CDT 2009


Nikolay Sivov schrieb:
> André Hentschel wrote:
>> related to Bug 4742
>> try2: fixed indentation
>> ---
>>  dlls/user32/edit.c |    6 ++++--
>>  1 files changed, 4 insertions(+), 2 deletions(-)
> ---
> 
> -            countA = strlen(textA) + 1;
> +            if(textA)
> +                countA = strlen(textA) + 1;
>         }
> ---
> 
> 
> Previous lines use 2 tabs and 4 spaces after.
> Also a test is needed, looks like bug contains already something to 
> start with.

I dont think we need a test, if textA is NULL then strlen crashes. Thats all.
Should i change the indentation of the code around?



More information about the wine-devel mailing list