diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2006-07-11 22:32:28 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2006-07-11 22:32:28 +0000 |
| commit | c87b6e76285a0f6da401a5de2aa7b096b9ec731a (patch) | |
| tree | ce12466219d5073f42779588295df091c6c274dc /libdwfl | |
| parent | 28ed895fdc303b2a793506bb1fcdd35d5fd14e70 (diff) | |
| download | elfutils-c87b6e76285a0f6da401a5de2aa7b096b9ec731a.tar.gz | |
Fix overflow in compare_modules return value.
Diffstat (limited to 'libdwfl')
| -rw-r--r-- | libdwfl/ChangeLog | 6 | ||||
| -rw-r--r-- | libdwfl/dwfl_module.c | 11 |
2 files changed, 15 insertions, 2 deletions
diff --git a/libdwfl/ChangeLog b/libdwfl/ChangeLog index 96fe56bc..bdfd12c2 100644 --- a/libdwfl/ChangeLog +++ b/libdwfl/ChangeLog @@ -1,3 +1,9 @@ +2006-07-11 Ulrich Drepper <drepper@redhat.com> + + * dwfl_module.c (compare_modules): Don't return GElf_Sxword value, + it can overflow the return value type. + Patch by Tim Moore <timoore@redhat.com>. + 2006-06-13 Roland McGrath <roland@redhat.com> * elf-from-memory.c (elf_from_remote_memory): Fix 32/64 typo. diff --git a/libdwfl/dwfl_module.c b/libdwfl/dwfl_module.c index 022bfea1..6f3aa849 100644 --- a/libdwfl/dwfl_module.c +++ b/libdwfl/dwfl_module.c @@ -1,5 +1,5 @@ /* Maintenance of module list in libdwfl. - Copyright (C) 2005 Red Hat, Inc. + Copyright (C) 2005, 2006 Red Hat, Inc. This file is part of Red Hat elfutils. Red Hat elfutils is free software; you can redistribute it and/or modify @@ -161,6 +161,7 @@ dwfl_report_module (Dwfl *dwfl, const char *name, } INTDEF (dwfl_report_module) + static int compare_modules (const void *a, const void *b) { @@ -170,7 +171,13 @@ compare_modules (const void *a, const void *b) return -1; if (m2 == NULL) return 1; - return (GElf_Sxword) (m1->low_addr - m2->low_addr); + + GElf_Sxword diff = m1->low_addr - m2->low_addr; + if (diff < 0) + return -1; + if (diff > 0) + return 1; + return 0; } |