[email protected] writes:

> Author: philip
> Date: Wed Aug 25 09:55:35 2010
> New Revision: 988956
>
> URL: http://svn.apache.org/viewvc?rev=988956&view=rev
> Log:
> Fix some uses of unitialised variables identified by valgrind.
>
> * subversion/libsvn_wc/update_editor.c
>   (add_directory, add_file): Set conflicted if svn_wc__db_read_info fails.
>
> Modified:
>     subversion/trunk/subversion/libsvn_wc/update_editor.c
>
> Modified: subversion/trunk/subversion/libsvn_wc/update_editor.c
> URL: 
> http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/update_editor.c?rev=988956&r1=988955&r2=988956&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/libsvn_wc/update_editor.c (original)
> +++ subversion/trunk/subversion/libsvn_wc/update_editor.c Wed Aug 25 09:55:35 
> 2010
> @@ -2357,6 +2357,7 @@ add_directory(const char *path,
>        svn_error_clear(err);
>        wc_kind = svn_wc__db_kind_unknown;
>        status = svn_wc__db_status_normal;
> +      conflicted = FALSE;
>  
>        versioned_locally_and_present = FALSE;
>      }
> @@ -3808,6 +3809,7 @@ add_file(const char *path,
>        svn_error_clear(err);
>        wc_kind = svn_wc__db_kind_unknown;
>        status = svn_wc__db_status_normal;
> +      conflicted = FALSE;
>  
>        versioned_locally_and_present = FALSE;
>      }

Could someone confirm that FALSE is correct here?

-- 
Philip

Reply via email to