Преглед изворни кода

Merge pull request #1055 from paperino1983/fix_coverity_CID349582_REVERSE_INULL

should fix *** CID 349582:  Null pointer dereferences  (REVERSE_INULL)
bel2125 пре 2 година
родитељ
комит
6ee7cda88d
1 измењених фајлова са 5 додато и 2 уклоњено
  1. 5 2
      src/civetweb.c

+ 5 - 2
src/civetweb.c

@@ -12823,11 +12823,14 @@ dav_lock_file(struct mg_connection *conn, const char *path)
 	int i;
 	int i;
 	uint64_t LOCK_DURATION_NS =
 	uint64_t LOCK_DURATION_NS =
 	    (uint64_t)(LOCK_DURATION_S) * (uint64_t)1000000000;
 	    (uint64_t)(LOCK_DURATION_S) * (uint64_t)1000000000;
-	struct twebdav_lock *dav_lock = conn->phys_ctx->webdav_lock;
+	struct twebdav_lock *dav_lock = NULL;
 
 
-	if (!path || !conn->dom_ctx || !conn->request_info.remote_user) {
+	if (!path || !conn
+		|| !conn->dom_ctx || !conn->request_info.remote_user || !conn->phys_ctx) {
 		return;
 		return;
 	}
 	}
+
+	dav_lock = conn->phys_ctx->webdav_lock;
 	mg_get_request_link(conn, link_buf, sizeof(link_buf));
 	mg_get_request_link(conn, link_buf, sizeof(link_buf));
 
 
 	/* const char *refresh = mg_get_header(conn, "If"); */
 	/* const char *refresh = mg_get_header(conn, "If"); */