From: Christopher Peplin <chris.peplin@rhubarbtech.com>
Date: Wed, 8 Jan 2014 22:27:54 +0000 (-0500)
Subject: Don't log incorrect negative response mode, just cap it at 0.
X-Git-Tag: 5.0.2~273^2~34
X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=6e08c6f35254273aa9750600be0effe6e7cf276e;p=apps%2Fagl-service-can-low-level.git

Don't log incorrect negative response mode, just cap it at 0.
---

diff --git a/src/obd2/obd2.c b/src/obd2/obd2.c
index c000aa3e..25b00e7c 100644
--- a/src/obd2/obd2.c
+++ b/src/obd2/obd2.c
@@ -205,8 +205,9 @@ DiagnosticResponse diagnostic_receive_can_frame(DiagnosticShims* shims,
                     handle->completed = true;
                 } else {
                     shims->log("Response was for a mode 0x%x request (pid 0x%x), not our mode 0x%x request (pid 0x%x)",
-                            response.mode - MODE_RESPONSE_OFFSET, response.pid,
-                            handle->request.mode, handle->request.pid);
+                            MAX(0, response.mode - MODE_RESPONSE_OFFSET),
+                            response.pid, handle->request.mode,
+                            handle->request.pid);
                     setup_receive_handle(handle);
                 }
             } else {