Re: [PATCH]Fix Segfault while detecting the implicit merge source

>> ctx, pool));
>> @@ -241,7 +242,7 @@
>> {
>> const char *path;
>> apr_hash_this(hi, (void *) &path, NULL, NULL);
>> - if (strcmp(path, copyfrom_path) != 0)
>> + if (copyfrom_path && strcmp(path, copyfrom_path) != 0)
>>
>
> Shouldn't this be?:
>
> if (copyfrom_path == NULL || strcmp(path, copyfrom_path) != 0)
>
>
No. It segfaults in strcmp when copyfrom_path is NULL.

With regards
Kamesh Jayachandran

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 15 19:39:07 2007