|
 |
fd304b |
From 458f73d58c24a7addce82bf1e8bfb8c2554ca458 Mon Sep 17 00:00:00 2001
|
|
|
fd304b |
From: Jean Delvare <jdelvare@suse.de>
|
|
|
fd304b |
Date: Wed, 14 Oct 2015 14:37:09 +0200
|
|
|
fd304b |
Subject: [PATCH 3/9] Fix error paths in mem_chunk
|
|
|
fd304b |
|
|
|
fd304b |
Use a common error path in function mem_chunk, to make sure it does
|
|
|
fd304b |
not leak memory and does not leave an opened file descriptor behind,
|
|
|
fd304b |
without duplicating the cleaning code.
|
|
|
fd304b |
---
|
|
|
fd304b |
CHANGELOG | 1 +
|
|
|
fd304b |
util.c | 24 ++++++++++--------------
|
|
|
fd304b |
2 files changed, 11 insertions(+), 14 deletions(-)
|
|
|
fd304b |
|
|
|
fd304b |
diff --git a/CHANGELOG b/CHANGELOG
|
|
|
fd304b |
index aa1c28f..c940c9f 100644
|
|
|
fd304b |
--- a/CHANGELOG
|
|
|
fd304b |
+++ b/CHANGELOG
|
|
|
fd304b |
@@ -3,6 +3,7 @@
|
|
|
fd304b |
* util.c: Avoid SIGBUS on mmap failure.
|
|
|
fd304b |
This fixes Savannah bug #46066:
|
|
|
fd304b |
https://savannah.nongnu.org/bugs/?46066
|
|
|
fd304b |
+ * util.c: Fix error paths in mem_chunk.
|
|
|
fd304b |
|
|
|
fd304b |
2015-10-01 Roy Franz <roy.franz@linaro.org>
|
|
|
fd304b |
|
|
|
fd304b |
diff --git a/util.c b/util.c
|
|
|
fd304b |
index 5795d02..f97ac0d 100644
|
|
|
fd304b |
--- a/util.c
|
|
|
fd304b |
+++ b/util.c
|
|
|
fd304b |
@@ -166,7 +166,7 @@ void *mem_chunk(off_t base, size_t len, const char *devmem)
|
|
|
fd304b |
if ((p = malloc(len)) == NULL)
|
|
|
fd304b |
{
|
|
|
fd304b |
perror("malloc");
|
|
|
fd304b |
- return NULL;
|
|
|
fd304b |
+ goto out;
|
|
|
fd304b |
}
|
|
|
fd304b |
|
|
|
fd304b |
#ifdef USE_MMAP
|
|
|
fd304b |
@@ -174,8 +174,7 @@ void *mem_chunk(off_t base, size_t len, const char *devmem)
|
|
|
fd304b |
{
|
|
|
fd304b |
fprintf(stderr, "%s: ", devmem);
|
|
|
fd304b |
perror("stat");
|
|
|
fd304b |
- free(p);
|
|
|
fd304b |
- return NULL;
|
|
|
fd304b |
+ goto err_free;
|
|
|
fd304b |
}
|
|
|
fd304b |
|
|
|
fd304b |
/*
|
|
|
fd304b |
@@ -186,8 +185,7 @@ void *mem_chunk(off_t base, size_t len, const char *devmem)
|
|
|
fd304b |
{
|
|
|
fd304b |
fprintf(stderr, "mmap: Can't map beyond end of file %s\n",
|
|
|
fd304b |
devmem);
|
|
|
fd304b |
- free(p);
|
|
|
fd304b |
- return NULL;
|
|
|
fd304b |
+ goto err_free;
|
|
|
fd304b |
}
|
|
|
fd304b |
|
|
|
fd304b |
#ifdef _SC_PAGESIZE
|
|
|
fd304b |
@@ -220,19 +218,17 @@ try_read:
|
|
|
fd304b |
{
|
|
|
fd304b |
fprintf(stderr, "%s: ", devmem);
|
|
|
fd304b |
perror("lseek");
|
|
|
fd304b |
- free(p);
|
|
|
fd304b |
- return NULL;
|
|
|
fd304b |
+ goto err_free;
|
|
|
fd304b |
}
|
|
|
fd304b |
|
|
|
fd304b |
- if (myread(fd, p, len, devmem) == -1)
|
|
|
fd304b |
- {
|
|
|
fd304b |
- free(p);
|
|
|
fd304b |
- return NULL;
|
|
|
fd304b |
- }
|
|
|
fd304b |
+ if (myread(fd, p, len, devmem) == 0)
|
|
|
fd304b |
+ goto out;
|
|
|
fd304b |
+
|
|
|
fd304b |
+err_free:
|
|
|
fd304b |
+ free(p);
|
|
|
fd304b |
+ p = NULL;
|
|
|
fd304b |
|
|
|
fd304b |
-#ifdef USE_MMAP
|
|
|
fd304b |
out:
|
|
|
fd304b |
-#endif
|
|
|
fd304b |
if (close(fd) == -1)
|
|
|
fd304b |
perror(devmem);
|
|
|
fd304b |
|
|
|
fd304b |
--
|
|
|
fd304b |
2.5.0
|
|
|
fd304b |
|