You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by da...@apache.org on 2007/08/04 02:30:01 UTC

svn commit: r562629 - in /apr/apr-util/branches/1.2.x: CHANGES dbm/apr_dbm_sdbm.c

Author: davi
Date: Fri Aug  3 17:30:01 2007
New Revision: 562629

URL: http://svn.apache.org/viewvc?view=rev&rev=562629
Log:
Revert revision 562627, the argument order is right.

Modified:
    apr/apr-util/branches/1.2.x/CHANGES
    apr/apr-util/branches/1.2.x/dbm/apr_dbm_sdbm.c

Modified: apr/apr-util/branches/1.2.x/CHANGES
URL: http://svn.apache.org/viewvc/apr/apr-util/branches/1.2.x/CHANGES?view=diff&rev=562629&r1=562628&r2=562629
==============================================================================
--- apr/apr-util/branches/1.2.x/CHANGES (original)
+++ apr/apr-util/branches/1.2.x/CHANGES Fri Aug  3 17:30:01 2007
@@ -1,8 +1,5 @@
 Changes with APR-util 1.2.9
 
-  *) Fix improper datum copy for dbm fetch (apr_dbm_fetch) operations
-     on SDBM databases.  [Davi Arnaut]
-
   *) Fix Solaris 2.8+ fdatasync() detection. The fdatasync() function
      is marked as part of the Realtime library functions.
      PR 37343.  [Davi Arnaut]

Modified: apr/apr-util/branches/1.2.x/dbm/apr_dbm_sdbm.c
URL: http://svn.apache.org/viewvc/apr/apr-util/branches/1.2.x/dbm/apr_dbm_sdbm.c?view=diff&rev=562629&r1=562628&r2=562629
==============================================================================
--- apr/apr-util/branches/1.2.x/dbm/apr_dbm_sdbm.c (original)
+++ apr/apr-util/branches/1.2.x/dbm/apr_dbm_sdbm.c Fri Aug  3 17:30:01 2007
@@ -140,7 +140,7 @@
     result_datum_t rd;
 
     CONVERT_DATUM(ckey, &key);
-    rv = APR_DBM_FETCH(dbm->file, rd, ckey);
+    rv = APR_DBM_FETCH(dbm->file, ckey, rd);
     RETURN_DATUM(pvalue, rd);
 
     REGISTER_CLEANUP(dbm, pvalue);