You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kg...@apache.org on 2020/12/09 15:49:33 UTC

[qpid-dispatch] branch dev-protocol-adaptors-2 updated: DISPATCH-1779: fix use-after free issue flagged by coverity

This is an automated email from the ASF dual-hosted git repository.

kgiusti pushed a commit to branch dev-protocol-adaptors-2
in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git


The following commit(s) were added to refs/heads/dev-protocol-adaptors-2 by this push:
     new 5ae5e65  DISPATCH-1779: fix use-after free issue flagged by coverity
5ae5e65 is described below

commit 5ae5e654b0ca184cdf130277176b096c43b59813
Author: Kenneth Giusti <kg...@apache.org>
AuthorDate: Wed Dec 9 10:30:43 2020 -0500

    DISPATCH-1779: fix use-after free issue flagged by coverity
---
 src/adaptors/http_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/adaptors/http_common.c b/src/adaptors/http_common.c
index 3d1d023..9034e37 100644
--- a/src/adaptors/http_common.c
+++ b/src/adaptors/http_common.c
@@ -555,7 +555,7 @@ static void _free_qdr_http_request_info(qdr_http_request_info_t* record)
     if (record->site) {
         free(record->site);
     }
-    for (qdr_http_method_status_t *item = DEQ_HEAD(record->detail); item; item = DEQ_NEXT(item)) {
+    for (qdr_http_method_status_t *item = DEQ_HEAD(record->detail); item; item = DEQ_HEAD(record->detail)) {
         _free_qdr_http_method_status(item);
     }
     free(record);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org