From c7266ff148ad801751c1e3d1e3c14d04b2ebee44 Mon Sep 17 00:00:00 2001 From: Tom Nicklin Date: Sat, 24 Mar 2018 18:58:15 +0000 Subject: [PATCH] Fixed issue where Valgrind reports points to uninitialised byte(s) in bind call. --- cansend.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cansend.c b/cansend.c index 3938029..ac49de3 100644 --- a/cansend.c +++ b/cansend.c @@ -102,6 +102,8 @@ int main(int argc, char **argv) return 1; } + bzero(&addr,sizeof(struct sockaddr_can)); //zero out the struct + addr.can_family = AF_CAN; addr.can_ifindex = ifr.ifr_ifindex; @@ -136,6 +138,8 @@ int main(int argc, char **argv) /* little (really a very little!) CPU usage. */ setsockopt(s, SOL_CAN_RAW, CAN_RAW_FILTER, NULL, 0); + // bzero(&addr,sizeof(struct sockaddr_can)); //zero out the struct + if (bind(s, (struct sockaddr *)&addr, sizeof(addr)) < 0) { perror("bind"); return 1;