summaryrefslogtreecommitdiffstats
path: root/lib/lzma
diff options
context:
space:
mode:
authorJeroen Hofstee <jeroen@myspectrum.nl>2014-06-10 23:37:23 +0200
committerTom Rini <trini@ti.com>2014-06-11 16:27:06 -0400
commit867abdac5effde660ac1ca9db8c43994edc01c09 (patch)
treebbf7519cf64f36070231fc1fe39192a2433a48a8 /lib/lzma
parentc84f9e0d42c499647b3cfbb6358d601b61a6b4cd (diff)
downloadblackbird-obmc-uboot-867abdac5effde660ac1ca9db8c43994edc01c09.tar.gz
blackbird-obmc-uboot-867abdac5effde660ac1ca9db8c43994edc01c09.zip
LzmaTools: don't self assign values
It seems the code tries to trick the compiler the argument is actually used. However compilers became too smart to fool them so easily an now warn. Gcc and clang don't seem to emit a warning when the argument is unused. If so it should be decorated with unused / (void). Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
Diffstat (limited to 'lib/lzma')
-rw-r--r--lib/lzma/LzmaTools.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/lzma/LzmaTools.c b/lib/lzma/LzmaTools.c
index 90d31cdcf8..cfc7cb02f7 100644
--- a/lib/lzma/LzmaTools.c
+++ b/lib/lzma/LzmaTools.c
@@ -34,8 +34,8 @@
#include <linux/string.h>
#include <malloc.h>
-static void *SzAlloc(void *p, size_t size) { p = p; return malloc(size); }
-static void SzFree(void *p, void *address) { p = p; free(address); }
+static void *SzAlloc(void *p, size_t size) { return malloc(size); }
+static void SzFree(void *p, void *address) { free(address); }
int lzmaBuffToBuffDecompress (unsigned char *outStream, SizeT *uncompressedSize,
unsigned char *inStream, SizeT length)
OpenPOWER on IntegriCloud