From: Romain Forlot <romain.forlot@iot.bzh>
Date: Thu, 20 Jul 2017 15:51:20 +0000 (+0200)
Subject: Fix subscribe response
X-Git-Tag: 4.0.0~24
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=9295346830bae064dddb4e0a12a69ddc9c120123;p=apps%2Flow-level-can-service.git

Fix subscribe response

This patch is "No signal(s) found for %s." return value is ret = -1.

Change-Id: Ia6f4f2db13beb7f9d3507d890383fc13cbddac0f
Signed-off-by: Yuichi Kusakabe <yuichi.kusakabe@jp.fujitsu.com>
---

diff --git a/low-can-binding/binding/low-can-cb.cpp b/low-can-binding/binding/low-can-cb.cpp
index e2ddf07..ec8b2b4 100644
--- a/low-can-binding/binding/low-can-cb.cpp
+++ b/low-can-binding/binding/low-can-cb.cpp
@@ -308,9 +308,12 @@ static int one_subscribe_unsubscribe(struct afb_req request, bool subscribe, con
 	openxc_DynamicField search_key = build_DynamicField(tag);
 	sf = utils::signals_manager_t::instance().find_signals(search_key);
 	if (sf.can_signals.empty() && sf.diagnostic_messages.empty())
+	{
 		AFB_NOTICE("No signal(s) found for %s.", tag.c_str());
+		ret = -1;
+	}
 	else
-		ret = subscribe_unsubscribe_signals(request, subscribe, sf, event_filter);
+		{ret = subscribe_unsubscribe_signals(request, subscribe, sf, event_filter);}
 
 	return ret;
 }