ubusd: fix a use after free bug
authorFelix Fietkau <nbd@openwrt.org>
Wed, 14 Sep 2011 15:09:19 +0000 (17:09 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 14 Sep 2011 15:09:19 +0000 (17:09 +0200)
ubusd_proto.c

index 902eb0f24b08a808e6a0f5e9889fecdb07813b14..821c24aca87164b8b2c7479a292cb9f82d9b2f2a 100644 (file)
@@ -207,6 +207,7 @@ static int ubusd_handle_lookup(struct ubus_client *cl, struct ubus_msg_buf *ub,
 
 static int ubusd_handle_invoke(struct ubus_client *cl, struct ubus_msg_buf *ub, struct blob_attr **attr)
 {
+       struct ubus_msg_buf *ub_new;
        struct ubus_object *obj = NULL;
        struct ubus_id *id;
        const char *method;
@@ -232,9 +233,10 @@ static int ubusd_handle_invoke(struct ubus_client *cl, struct ubus_msg_buf *ub,
                blob_put(&b, UBUS_ATTR_DATA, blob_data(attr[UBUS_ATTR_DATA]),
                         blob_len(attr[UBUS_ATTR_DATA]));
 
+       ub_new = ubus_reply_from_blob(ub, true);
        ubus_msg_free(ub);
+       ub = ub_new;
 
-       ub = ubus_reply_from_blob(ub, true);
        if (!ub)
                return UBUS_STATUS_NO_DATA;