]> granicus.if.org Git - apache/commitdiff
* modules/slotmem/mod_slotmem_shm.c (restore_slotmem): Remove
authorJoe Orton <jorton@apache.org>
Fri, 5 Oct 2018 10:17:18 +0000 (10:17 +0000)
committerJoe Orton <jorton@apache.org>
Fri, 5 Oct 2018 10:17:18 +0000 (10:17 +0000)
  redundant assignment (clang warning), the apr_file_eof(fp)=>APR_EOF
  case assigns rv to APR_EOF and then to APR_SUCCESS after already.

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

modules/slotmem/mod_slotmem_shm.c

index 7dffef19fcec333ff75cbfcb7cd0d2006263c0fc..6dda8f65fec3d9444f54d9983c55e415f0fb2226 100644 (file)
@@ -214,7 +214,6 @@ static apr_status_t restore_slotmem(sharedslotdesc_t *desc,
         if (rv == APR_SUCCESS) {
             rv = apr_file_read_full(fp, ptr, nbytes, NULL);
             if (rv == APR_SUCCESS || rv == APR_EOF) {
-                rv = APR_SUCCESS;   /* for successful return @ EOF */
                 /*
                  * if at EOF, don't bother checking md5
                  *  - backwards compatibility