You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by rh...@apache.org on 2014/03/27 18:17:24 UTC

svn commit: r1582409 - /subversion/trunk/subversion/libsvn_fs_x/lock.c

Author: rhuijben
Date: Thu Mar 27 17:17:24 2014
New Revision: 1582409

URL: http://svn.apache.org/r1582409
Log:
Following up on r1582370 which was a followup to r1577280, apply the same
usage of uninitialized variable fix to fsx.

* subversion/libsvn_fs_fs/lock.c
  (unlock_body): Properly initialize struct before using members from it.
    Without this patch info.fs_err will be read before setting a few lines
    later... and then the total struct copied in the array.

Modified:
    subversion/trunk/subversion/libsvn_fs_x/lock.c

Modified: subversion/trunk/subversion/libsvn_fs_x/lock.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_x/lock.c?rev=1582409&r1=1582408&r2=1582409&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_x/lock.c (original)
+++ subversion/trunk/subversion/libsvn_fs_x/lock.c Thu Mar 27 17:17:24 2014
@@ -1045,7 +1045,7 @@ unlock_body(void *baton, apr_pool_t *poo
       const svn_sort__item_t *item = &APR_ARRAY_IDX(ub->targets, i,
                                                     svn_sort__item_t);
       const char *token = item->value;
-      struct unlock_info_t info;
+      struct unlock_info_t info = { 0 };
 
       svn_pool_clear(iterpool);