mod_dav: PROPPATCH delete (svn propdel) silently discards errors. PR 53525

Submitted by Arwin Arni <arwin collab.net>

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1476627 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/CHANGES b/CHANGES
index 9d40d2498a065acf082cc865520691f5b2fa9c5a..88b2af112c1b513587cc87fb21a9ff473dfca0d5 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache 2.5.0
 
+  *) mod_dav: PROPPATCH delete (svn propdel) silently discards errors.
+     PR 53525 [Arwin Arni <arwin collab.net>]
+
   *) mod_dav: Ensure URI is correctly uriencoded on return. PR 54611
      [Timothy Wood <tjw omnigroup.com>]
 

diff --git a/modules/dav/main/props.c b/modules/dav/main/props.c
index d87dd61131657c57e334bd70915c35765cb01428..19c7b11aef1b400892804076631f6201c4336da6 100644 (file)
--- a/modules/dav/main/props.c
+++ b/modules/dav/main/props.c
@@ -1042,11 +1042,9 @@ DAV_DECLARE_NONSTD(void) dav_prop_exec(dav_prop_ctx *ctx)
         else if (ctx->operation == DAV_PROP_OP_DELETE) {
 
             /*
-            ** Delete the property. Ignore errors -- the property is there, or
-            ** we are deleting it for a second time.
+            ** Delete the property.
             */
-            /* ### but what about other errors? */
-            (void) (*propdb->db_hooks->remove)(propdb->db, &name);
+            err = (*propdb->db_hooks->remove)(propdb->db, &name);
         }
     }