From: Romain Forlot <romain.forlot@iot.bzh>
Date: Wed, 7 Jun 2017 13:42:21 +0000 (+0200)
Subject: Fix: read 2 times for 1 incoming CAN frame.
X-Git-Tag: 5.0.2~103
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=995fc3b8b8e51f0adfa2dac06286ede84f05a43a;p=apps%2Fagl-service-can-low-level.git

Fix: read 2 times for 1 incoming CAN frame.

Change-Id: I7be7b5e704b3cb081e1dbd73a6ba7f88a2bbc62f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
---

diff --git a/CAN-binder/low-can-binding/binding/low-can-cb.cpp b/CAN-binder/low-can-binding/binding/low-can-cb.cpp
index fbe6e196..bc94eedf 100644
--- a/CAN-binder/low-can-binding/binding/low-can-cb.cpp
+++ b/CAN-binder/low-can-binding/binding/low-can-cb.cpp
@@ -242,13 +242,13 @@ static int subscribe_unsubscribe_can_signals(struct afb_req request, bool subscr
 			 can_subscription = std::make_shared<low_can_subscription_t>(low_can_subscription_t(event_filter));
 			if(can_subscription->create_rx_filter(sig) < 0)
 				{return -1;}
+			if(add_to_event_loop(can_subscription) < 0)
+				{return -1;}
 		}
 
 		if(subscribe_unsubscribe_signal(request, subscribe, can_subscription, s) < 0)
 			{return -1;}
 
-		if(add_to_event_loop(can_subscription) < 0)
-			{return -1;}
 		rets++;
 		DEBUG(binder_interface, "%s: signal: %s subscribed", __FUNCTION__, sig->get_name().c_str());
 	}