You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by ro...@apache.org on 2006/10/09 15:59:33 UTC

svn commit: r454383 - in /apr/apr-util/trunk: CHANGES dbm/sdbm/sdbm.c

Author: rooneg
Date: Mon Oct  9 06:59:33 2006
New Revision: 454383

URL: http://svn.apache.org/viewvc?view=rev&rev=454383
Log:
Fix precedence problem in sdbm dbm back end.

Submitted by: Larry Cipriani <lvc lucent.com>
PR: 40659

* dbm/sdbm/sdbm.c
  (getnext): Actually check error from apr_file_seek.

* CHANGES: Note change.

Modified:
    apr/apr-util/trunk/CHANGES
    apr/apr-util/trunk/dbm/sdbm/sdbm.c

Modified: apr/apr-util/trunk/CHANGES
URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/CHANGES?view=diff&rev=454383&r1=454382&r2=454383
==============================================================================
--- apr/apr-util/trunk/CHANGES (original)
+++ apr/apr-util/trunk/CHANGES Mon Oct  9 06:59:33 2006
@@ -1,5 +1,8 @@
 Changes with APR-util 1.3.0
 
+  *) Fix precedence problem in error checking for sdbm dbm back end.
+     PR 40659 [Larry Cipriani <lvc lucent.com>]
+
   *) Add an apr_reslist_acquired_count, for determining how many outstanding
      resources there are in a reslist.  [Ryan Phillips <ryan trolocsis.com>]
 

Modified: apr/apr-util/trunk/dbm/sdbm/sdbm.c
URL: http://svn.apache.org/viewvc/apr/apr-util/trunk/dbm/sdbm/sdbm.c?view=diff&rev=454383&r1=454382&r2=454383
==============================================================================
--- apr/apr-util/trunk/dbm/sdbm/sdbm.c (original)
+++ apr/apr-util/trunk/dbm/sdbm/sdbm.c Mon Oct  9 06:59:33 2006
@@ -560,8 +560,8 @@
         db->keyptr = 0;
         if (db->pagbno != db->blkptr++) {
             apr_off_t off = OFF_PAG(db->blkptr);
-            if ((status = apr_file_seek(db->pagf, APR_SET, &off) 
-                        != APR_SUCCESS))
+            if ((status = apr_file_seek(db->pagf, APR_SET, &off))
+                        != APR_SUCCESS)
                 return status;
         }