diff options
| author |   Matt Ranostay <matt.ranostay@konsulko.com> | 2019-11-27 22:56:30 -0800 |
|---|---|---|
| committer | Matt Ranostay <matt.ranostay@konsulko.com> | 2019-11-28 17:15:03 -0800 |
| commit | dbd47e0c6cf83168e5d4f4f7b850d9bc728a175f (patch) | |
| tree | e10ba8369512ce36519f5ca9cde6f375a1f40278 | |
| parent | 6a1a315f3b05309a6bb385407f1ba7a24293e243 (diff) | |
binding: pbap: add missing return after afb_req_fail callicefish_8.99.5icefish_8.99.4icefish_8.99.3icefish_8.99.2icefish/8.99.5icefish/8.99.4icefish/8.99.3icefish/8.99.28.99.58.99.48.99.38.99.2
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>
| -rw-r--r-- | binding/bluetooth-pbap-binding.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/binding/bluetooth-pbap-binding.c b/binding/bluetooth-pbap-binding.c index 13421e5..8274f05 100644 --- a/binding/bluetooth-pbap-binding.c +++ b/binding/bluetooth-pbap-binding.c @@ -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"); |