diff options
| author | Jon Loeliger <jdl@freescale.com> | 2008-09-25 11:02:17 -0500 |
|---|---|---|
| committer | Jon Loeliger <jdl@jdl.com> | 2008-09-25 15:45:48 -0500 |
| commit | b236893fc4bc173877adfa4270b14d7d7209fb10 (patch) | |
| tree | ce74c75992065d679efe212c161d29b389bb66c2 /libfdt/fdt.c | |
| parent | 8daae14b742a48078c77fca3ccb24cd376232bc1 (diff) | |
| download | dtc-b236893fc4bc173877adfa4270b14d7d7209fb10.tar.gz | |
Fix an overflow case in fdt_offset_ptr() detected by GCC 4.3.
Using Gcc 4.3 detected this problem:
../dtc/libfdt/fdt.c: In function 'fdt_next_tag':
../dtc/libfdt/fdt.c:82: error: assuming signed overflow does not
occur when assuming that (X + c) < X is always false
To fix the problem, treat the offset as an unsigned int.
The problem report and proposed fix were provided
by Steve Papacharalambous <stevep@freescale.com>.
Signed-off-by: Jon Loeliger <jdl@freescale.com>
Diffstat (limited to 'libfdt/fdt.c')
| -rw-r--r-- | libfdt/fdt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libfdt/fdt.c b/libfdt/fdt.c index 2acaec5..37085c1 100644 --- a/libfdt/fdt.c +++ b/libfdt/fdt.c @@ -74,7 +74,7 @@ int fdt_check_header(const void *fdt) return 0; } -const void *fdt_offset_ptr(const void *fdt, int offset, int len) +const void *fdt_offset_ptr(const void *fdt, int offset, unsigned int len) { const char *p; |