File 0200-9pfs-drop-useless-check-in-pdu_free.patch of Package qemu-testsuite.6353

Overview Repositories Revisions Requests Users Attributes Meta

File 0200-9pfs-drop-useless-check-in-pdu_free.patch of Package qemu-testsuite.6353

From ed5a05ba72c8721adee38bc42edf10fc20550695 Mon Sep 17 00:00:00 2001
From: Greg Kurz <groug@kaod.org>
Date: Mon, 17 Oct 2016 14:13:58 +0200
Subject: [PATCH] 9pfs: drop useless check in pdu_free()

Out of the three users of pdu_free(), none ever passes a NULL pointer to
this function.

Signed-off-by: Greg Kurz <groug@kaod.org>
(cherry picked from commit 6868a420c519d74926ea814d48f6ce9beda35b98)
[BR: Fix and/or infrastructure for BSC#1020427 CVE-2016-9602]
Signed-off-by: Bruce Rogers <brogers@suse.com>
---
 hw/9pfs/9p.c | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c
index e3df928160..43fc44be8b 100644
--- a/hw/9pfs/9p.c
+++ b/hw/9pfs/9p.c
@@ -604,16 +604,14 @@ V9fsPDU *pdu_alloc(V9fsState *s)
 
 void pdu_free(V9fsPDU *pdu)
 {
-    if (pdu) {
-        V9fsState *s = pdu->s;
-        /*
-         * Cancelled pdu are added back to the freelist
-         * by flush request .
-         */
-        if (!pdu->cancelled) {
-            QLIST_REMOVE(pdu, next);
-            QLIST_INSERT_HEAD(&s->free_list, pdu, next);
-        }
+    V9fsState *s = pdu->s;
+    /*
+     * Cancelled pdu are added back to the freelist
+     * by flush request .
+     */
+    if (!pdu->cancelled) {
+        QLIST_REMOVE(pdu, next);
+        QLIST_INSERT_HEAD(&s->free_list, pdu, next);
     }
 }

Places

File 0200-9pfs-drop-useless-check-in-pdu_free.patch of Package qemu-testsuite.6353

Places