diff options
| author | Jon Loeliger <jdl@jdl.com> | 2008-10-03 11:12:33 -0500 |
|---|---|---|
| committer | Jon Loeliger <jdl@jdl.com> | 2008-10-03 11:12:33 -0500 |
| commit | 879e4d2590b50d63f82c3c3652bc3c7900591f1c (patch) | |
| tree | 360150a0a9f7b94f455ace32fc5145898d4c4b2f /Makefile.convert-dtsv0 | |
| parent | 68f98d7b8aa41fab175daf9f1bcb2a5bc22dbc90 (diff) | |
| download | dtc-879e4d2590b50d63f82c3c3652bc3c7900591f1c.tar.gz | |
Implement and use an xstrdup() function
Many places in dtc use strdup(), but none of them actually check the
return value to see if the implied allocation succeeded. This is a
potential bug, which we fix in the patch below by replacing strdup()
with an xstrdup() which in analogy to xmalloc() will quit with a fatal
error if the allocation fails.
I felt the introduciton of util.[ch] was a better choice
for utility oriented code than directly using srcpos.c
for the new string function.
This patch is a re-factoring of Dave Gibson's similar patch.
Signed-off-by: Jon Loeliger <jdl@freescale.com>
Diffstat (limited to 'Makefile.convert-dtsv0')
| -rw-r--r-- | Makefile.convert-dtsv0 | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Makefile.convert-dtsv0 b/Makefile.convert-dtsv0 index a3f74c9..08ea40a 100644 --- a/Makefile.convert-dtsv0 +++ b/Makefile.convert-dtsv0 @@ -5,7 +5,8 @@ # CONVERT_SRCS = \ - srcpos.c + srcpos.c \ + util.c CONVERT_GEN_SRCS = convert-dtsv0-lexer.lex.c |