diff options
| author | Edgar E. Iglesias <edgar@axis.com> | 2012-10-09 12:21:17 +0200 |
|---|---|---|
| committer | Edgar E. Iglesias <edgar@axis.com> | 2012-10-15 09:33:10 +0200 |
| commit | fd4b4ef0d4fbaf2039fac61764421e47cac2ca39 (patch) | |
| tree | d8ff687551501840c136b5693afd3ceae7ee0e86 /sysdeps | |
| parent | cc77b0e0e735d598f4bc746f8e79b6cd342bd606 (diff) | |
| download | ltrace-fd4b4ef0d4fbaf2039fac61764421e47cac2ca39.tar.gz | |
Allow extension tags in proc_find_dynamic_entry_addr
Remove the check for generic tags allowing the caller
to pass extension tags. I couldn't find a way to strictly
validate the tag nr, furthermore the seek tag is internally
generated so I simply dropped the check (beyond checking for
positive values).
One way possible could be to check for:
(tag >= DT_NULL && tag < DT_NUM) || (tag >= DT_LOOS && tag <= DT_HIPROC)
But I'm not sure thats OK.
Signed-off-by: Edgar E. Iglesias <edgar@axis.com>
Diffstat (limited to 'sysdeps')
| -rw-r--r-- | sysdeps/linux-gnu/proc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sysdeps/linux-gnu/proc.c b/sysdeps/linux-gnu/proc.c index 450177c..cc322fa 100644 --- a/sysdeps/linux-gnu/proc.c +++ b/sysdeps/linux-gnu/proc.c @@ -323,7 +323,7 @@ proc_find_dynamic_entry_addr(struct Process *proc, arch_addr_t src_addr, { debug(DEBUG_FUNCTION, "find_dynamic_entry()"); - if (ret == NULL || src_addr == 0 || d_tag < 0 || d_tag > DT_NUM) + if (ret == NULL || src_addr == 0 || d_tag < 0) return -1; int i = 0; |