diff options
author | Krzysztof Halasa <khc@pm.waw.pl> | 2011-01-06 16:23:01 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2011-01-07 10:21:22 +0100 |
commit | c5baa0edc4a70c2f44cb7422e2b47ab5c96570ed (patch) | |
tree | d45c280198754e96276f6a633a206233504790b3 /scripts | |
parent | c99d28ca568a6e0d713c42bfbb9cd5310588d857 (diff) | |
download | barebox-c5baa0edc4a70c2f44cb7422e2b47ab5c96570ed.tar.gz barebox-c5baa0edc4a70c2f44cb7422e2b47ab5c96570ed.tar.xz |
Fix error handling with malloc, memalign etc. Introduce xmemalign().
The idea is to panic() when there is no memory available for normal
operation. Exception: code which can consume arbitrary amount of RAM
(example: files allocated in ramfs) must report error instead of panic().
This patch also fixes code which didn't check for NULL from malloc() etc.
Usage: malloc(), memalign() return NULL when out of RAM.
xmalloc(), xmemalign() always return non-NULL or panic().
Signed-off-by: Krzysztof HaĆasa <khc@pm.waw.pl>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'scripts')
-rw-r--r-- | scripts/basic/docproc.c | 7 | ||||
-rw-r--r-- | scripts/mod/modpost.c | 2 |
2 files changed, 6 insertions, 3 deletions
diff --git a/scripts/basic/docproc.c b/scripts/basic/docproc.c index 0d4f5e7d74..9f6535d18f 100644 --- a/scripts/basic/docproc.c +++ b/scripts/basic/docproc.c @@ -122,8 +122,11 @@ int symfilecnt = 0; void add_new_symbol(struct symfile *sym, char * symname) { - sym->symbollist = - realloc(sym->symbollist, (sym->symbolcnt + 1) * sizeof(char *)); + sym->symbollist = realloc(sym->symbollist, (sym->symbolcnt + 1) * sizeof(char *)); + if (!sym->symbollist) { + fprintf(stderr, "docproc: out of memory\n"); + exit(1); + } sym->symbollist[sym->symbolcnt++].name = strdup(symname); } diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 4075c35d37..08b75b67f1 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -1317,7 +1317,7 @@ void buf_write(struct buffer *buf, const char *s, int len) { if (buf->size - buf->pos < len) { buf->size += len + SZ; - buf->p = realloc(buf->p, buf->size); + buf->p = NOFAIL(realloc(buf->p, buf->size)); } strncpy(buf->p + buf->pos, s, len); buf->pos += len; |