lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 12 Jan 2007 16:38:52 +0000 From: Russell King <rmk+lkml@....linux.org.uk> To: David Brownell <david-b@...bell.net> Cc: Linux Kernel list <linux-kernel@...r.kernel.org>, rusty@...tcorp.com.au Subject: Re: [patch 2.6.20-rc4-git] remove modpost false warnings on ARM On Fri, Jan 12, 2007 at 08:31:36AM -0800, David Brownell wrote: > Index: at91/scripts/mod/modpost.c > =================================================================== > --- at91.orig/scripts/mod/modpost.c 2007-01-11 22:51:49.000000000 -0800 > +++ at91/scripts/mod/modpost.c 2007-01-12 04:20:00.000000000 -0800 > @@ -679,6 +679,26 @@ static Elf_Sym *find_elf_symbol(struct e > } > > /* > + * If there's no name there, ignore it; likewise, ignore it if it's > + * one of the magic symbols emitted used by current ARM tools. > + * > + * Otherwise if find_symbols_between() returns those symbols, they'll > + * fail the whitelist tests and cause lots of false alarms ... fixable > + * only by shrinking __exit and __init sections into __text, bloating > + * the kernel (which is especially evil on embedded platforms). > + */ > +static int is_valid_name(struct elf_info *elf, Elf_Sym *sym) > +{ > + const char *name = elf->strtab + sym->st_name; > + > + if (!name || !strlen(name)) > + return 0; > + if (strcmp(name, "$a") == 0 || strcmp(name, "$d") == 0) > + return 0; A more correct test would be that found in kallsyms.c: /* * This ignores the intensely annoying "mapping symbols" found * in ARM ELF files: $a, $t and $d. */ static inline int is_arm_mapping_symbol(const char *str) { return str[0] == '$' && strchr("atd", str[1]) && (str[2] == '\0' || str[2] == '.'); } Suggest that code is re-used here (as well as in other tools such as oprofile, readprofile, etc.) -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists