You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by cm...@apache.org on 2011/01/07 21:01:03 UTC

svn commit: r1056480 - /subversion/trunk/subversion/libsvn_fs_base/lock.c

Author: cmpilato
Date: Fri Jan  7 20:01:03 2011
New Revision: 1056480

URL: http://svn.apache.org/viewvc?rev=1056480&view=rev
Log:
Handle some overlooked error returns.  Followup to r1056468.

* subversion/libsvn_fs_base/lock.c
  (svn_fs_base__get_locks): Handle return value from get_locks_func()
    and svn_io_file_seek().

Reported by: blair

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

Modified: subversion/trunk/subversion/libsvn_fs_base/lock.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_base/lock.c?rev=1056480&r1=1056479&r2=1056480&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_base/lock.c (original)
+++ subversion/trunk/subversion/libsvn_fs_base/lock.c Fri Jan  7 20:01:03 2011
@@ -468,7 +468,7 @@ svn_fs_base__get_locks(svn_fs_t *fs,
   SVN_ERR(svn_fs_base__retry_txn(fs, txn_body_get_locks, &args, FALSE, pool));
 
   /* Rewind the spool file, then re-read it, calling GET_LOCKS_FUNC(). */
-  svn_io_file_seek(args.spool_file, APR_SET, &offset, pool);
+  SVN_ERR(svn_io_file_seek(args.spool_file, APR_SET, &offset, pool));
   stream = svn_stream_from_aprfile2(args.spool_file, FALSE, pool);
 
   while (1)
@@ -502,7 +502,7 @@ svn_fs_base__get_locks(svn_fs_t *fs,
       /* Parse the skel into a lock, and notify the caller. */
       lock_skel = svn_skel__parse(skel_buf, skel_len, iterpool);
       SVN_ERR(svn_fs_base__parse_lock_skel(&lock, lock_skel, iterpool));
-      get_locks_func(get_locks_baton, lock, iterpool);
+      SVN_ERR(get_locks_func(get_locks_baton, lock, iterpool));
     }
 
   SVN_ERR(svn_stream_close(stream));