From df7ba78142677bde559d5ee48fea86e58c2020ce Mon Sep 17 00:00:00 2001
From: Marcus Winter <marcus.winter.ext@siemens.com>
Date: Thu, 11 Feb 2016 10:07:38 +0100
Subject: [PATCH] mtapi_network_c: buffer pop operations set values to zero on failure

---
 mtapi_network_c/src/embb_mtapi_network_buffer.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/mtapi_network_c/src/embb_mtapi_network_buffer.c b/mtapi_network_c/src/embb_mtapi_network_buffer.c
index 399a530..76ef1b0 100644
--- a/mtapi_network_c/src/embb_mtapi_network_buffer.c
+++ b/mtapi_network_c/src/embb_mtapi_network_buffer.c
@@ -107,6 +107,7 @@ int embb_mtapi_network_buffer_pop_front_int8(
   embb_mtapi_network_buffer_t * that,
   int8_t * value) {
   if (that->position + 1 > that->size) {
+    *value = 0;
     return 0;
   }
   memcpy(value, that->data + that->position, 1);
@@ -118,6 +119,7 @@ int embb_mtapi_network_buffer_pop_front_int16(
   embb_mtapi_network_buffer_t * that,
   int16_t * value) {
   if (that->position + 2 > that->size) {
+    *value = 0;
     return 0;
   }
   memcpy(value, that->data + that->position, 2);
@@ -129,6 +131,7 @@ int embb_mtapi_network_buffer_pop_front_int32(
   embb_mtapi_network_buffer_t * that,
   int32_t * value) {
   if (that->position + 4 > that->size) {
+    *value = 0;
     return 0;
   }
   memcpy(value, that->data + that->position, 4);
@@ -141,6 +144,7 @@ int embb_mtapi_network_buffer_pop_front_rawdata(
   int32_t size,
   void * rawdata) {
   if (that->position + size > that->size) {
+    memset(rawdata, 0, (size_t)size);
     return 0;
   }
   memcpy(rawdata, that->data + that->position, (size_t)size);
--
libgit2 0.26.0