[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[qemu-xen staging-4.14] chardev/tcp: Fix error message double free error



commit 54e74a45a58912dd7705d53e965e9092a0631aa5
Author:     lichun <lichun@xxxxxxxxxxxxx>
AuthorDate: Mon Jun 22 05:30:17 2020 +0800
Commit:     Michael Roth <mdroth@xxxxxxxxxxxxxxxxxx>
CommitDate: Wed Sep 2 19:06:19 2020 -0500

    chardev/tcp: Fix error message double free error
    
    Errors are already freed by error_report_err, so we only need to call
    error_free when that function is not called.
    
    Cc: qemu-stable@xxxxxxxxxx
    Signed-off-by: lichun <lichun@xxxxxxxxxxxxx>
    Message-Id: <20200621213017.17978-1-lichun@xxxxxxxxxxxxx>
    Reviewed-by: Markus Armbruster <armbru@xxxxxxxxxx>
    [Commit message improved, cc: qemu-stable]
    Signed-off-by: Markus Armbruster <armbru@xxxxxxxxxx>
    (cherry picked from commit ed4e0d2ef140aef255d67eec30767e5fcd949f58)
    Signed-off-by: Michael Roth <mdroth@xxxxxxxxxxxxxxxxxx>
---
 chardev/char-socket.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/chardev/char-socket.c b/chardev/char-socket.c
index e5ee685f8c..58917870cd 100644
--- a/chardev/char-socket.c
+++ b/chardev/char-socket.c
@@ -142,6 +142,8 @@ static void check_report_connect_error(Chardev *chr,
                           "Unable to connect character device %s: ",
                           chr->label);
         s->connect_err_reported = true;
+    } else {
+        error_free(err);
     }
     qemu_chr_socket_restart_timer(chr);
 }
@@ -1083,7 +1085,6 @@ static void qemu_chr_socket_connected(QIOTask *task, void 
*opaque)
     if (qio_task_propagate_error(task, &err)) {
         tcp_chr_change_state(s, TCP_CHARDEV_STATE_DISCONNECTED);
         check_report_connect_error(chr, err);
-        error_free(err);
         goto cleanup;
     }
 
--
generated by git-patchbot for /home/xen/git/qemu-xen.git#staging-4.14



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.