aboutsummaryrefslogtreecommitdiffstats
path: root/tools/firmware-utils/src
diff options
context:
space:
mode:
authorMathias Kresin <dev@kresin.me>2017-06-26 19:22:52 +0200
committerMathias Kresin <dev@kresin.me>2017-06-26 20:08:36 +0200
commit06741411e020bb28160ddd42149d6a489800dbc7 (patch)
tree353b36569c688cac8ac14e2ea16702f5ac5f8600 /tools/firmware-utils/src
parentf33de8023228d7d6545ff12c4f2ef8febb486f37 (diff)
downloadupstream-06741411e020bb28160ddd42149d6a489800dbc7.tar.gz
upstream-06741411e020bb28160ddd42149d6a489800dbc7.tar.bz2
upstream-06741411e020bb28160ddd42149d6a489800dbc7.zip
firmware-utils: fix dgn3500sum compiler warnings
The sum variable need to be initialised, otherwise it will points to random stack memory and a bogus image checksum might be calculated. While at it, fix the segfault in case the product region code isn't specified and enable compiler warnings which had revealed all the code issues. Signed-off-by: Mathias Kresin <dev@kresin.me>
Diffstat (limited to 'tools/firmware-utils/src')
-rw-r--r--tools/firmware-utils/src/dgn3500sum.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/tools/firmware-utils/src/dgn3500sum.c b/tools/firmware-utils/src/dgn3500sum.c
index 00a0c5f837..eb80e6c01b 100644
--- a/tools/firmware-utils/src/dgn3500sum.c
+++ b/tools/firmware-utils/src/dgn3500sum.c
@@ -103,7 +103,7 @@ int main(int argc, char** argv)
unsigned long start, i;
char *endptr, *buffer, *p;
int count; // size of file in bytes
- unsigned short sum, sum1;
+ unsigned short sum = 0, sum1 = 0;
char sumbuf[9];
if(argc < 3) {
@@ -117,7 +117,7 @@ int main(int argc, char** argv)
printf("ERROR: File not writeable!\n");
return 1;
}
- if(argc = 4)
+ if(argc == 4)
{
printf("%s: PID type: %s\n", argv[0], argv[3]);
if(strcmp(argv[3], "DE")==0)
@@ -147,7 +147,6 @@ int main(int argc, char** argv)
start = strtol(argv[2], &endptr, 16);
p = buffer+start;
- sum1 = 0;
for(i = 0; i < count - start; i++)
{
sum1 += p[i];