# HG changeset patch # User Richard Lowe # Date 1375824116 0 # Node ID f40648257ac63315be77d0906634dfa5b9afeebc # Parent 6f291eb437b921a07c0fd05761c90d30284456c2 4000 pvs can't deal with extended sections Reviewed by: Jason King Reviewed by: Josef 'Jeff' Sipek Approved by: Robert Mustacchi diff -r 6f291eb437b9 -r f40648257ac6 usr/src/cmd/sgs/pvs/common/pvs.c --- a/usr/src/cmd/sgs/pvs/common/pvs.c Tue Aug 06 17:09:22 2013 -0400 +++ b/usr/src/cmd/sgs/pvs/common/pvs.c Tue Aug 06 21:21:56 2013 +0000 @@ -1090,6 +1090,7 @@ for (; optind < argc; optind++) { int derror = 0, nerror = 0, err; const char *file = argv[optind]; + size_t shnum = 0; if ((var = open(file, O_RDONLY)) == -1) { err = errno; @@ -1154,7 +1155,13 @@ * as elf_begin has already gone through all the overhead we * might as well set up the cache for every section. */ - if ((cache = calloc(ehdr.e_shnum, sizeof (Cache))) == NULL) { + if (elf_getshdrnum(elf, &shnum) == -1) { + (void) fprintf(stderr, MSG_ORIG(MSG_ELF_GETSHDRNUM), + cname, file, elf_errmsg(elf_errno())); + exit(1); + } + + if ((cache = calloc(shnum, sizeof (Cache))) == NULL) { int err = errno; (void) fprintf(stderr, MSG_INTL(MSG_SYS_MALLOC), cname, file, strerror(err)); diff -r 6f291eb437b9 -r f40648257ac6 usr/src/cmd/sgs/pvs/common/pvs.msg --- a/usr/src/cmd/sgs/pvs/common/pvs.msg Tue Aug 06 17:09:22 2013 -0400 +++ b/usr/src/cmd/sgs/pvs/common/pvs.msg Tue Aug 06 21:21:56 2013 +0000 @@ -77,6 +77,7 @@ @ MSG_ELF_GETSCN "%s: %s: elf_getscn: %s\n" @ MSG_ELF_GETSHDR "%s: %s: elf_getshdr: %s\n" @ MSG_ELF_GETDATA "%s: %s: elf_getdata: %s\n" +@ MSG_ELF_GETSHDRNUM "%s: %s: elf_getshdrnum: %s\n" @ MSG_STR_EMPTY "" @ MSG_STR_OPTIONS "CdI:lnorsvN:"