binding: pbap: add missing return after afb_req_fail call 20/23120/1 8.99.2 8.99.3 8.99.4 8.99.5 icefish/8.99.2 icefish/8.99.3 icefish/8.99.4 icefish/8.99.5 icefish_8.99.2 icefish_8.99.3 icefish_8.99.4 icefish_8.99.5
authorMatt Ranostay <matt.ranostay@konsulko.com>
Thu, 28 Nov 2019 06:56:30 +0000 (22:56 -0800)
committerMatt Ranostay <matt.ranostay@konsulko.com>
Fri, 29 Nov 2019 01:15:03 +0000 (17:15 -0800)
To avoid calling afb_req_success after failure return after
the afb_req_fail call in contacts verb.

Bug-AGL: SPEC-2630
Change-Id: I2d521d7c04a4be7dd49e8a63cf1ac267de636de8
Signed-off-by: Matt Ranostay <matt.ranostay@konsulko.com>
binding/bluetooth-pbap-binding.c

index 13421e5..8274f05 100644 (file)
@@ -370,6 +370,7 @@ void contacts(afb_req_t request)
                jresp = json_tokener_parse(cached);
        } else {
                afb_req_fail(request, "no imported contacts", NULL);
+               return;
        }
 
        afb_req_success(request, jresp, "contacts");