From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from kwanyin.sergiodj.net (kwanyin.sergiodj.net [158.69.185.54]) by sourceware.org (Postfix) with ESMTPS id 0BFE9395A837 for ; Wed, 20 May 2020 16:10:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 0BFE9395A837 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [binutils-gdb] gdb/testsuite: check mmap ret val against MAP_FAILED From: gdb-buildbot@sergiodj.net To: gdb-testers@sourceware.org Message-Id: <41977d16e4ee5b9ad01abf2cfce6edbfb6d79541@gdb-build> Date: Wed, 20 May 2020 12:10:32 -0400 X-Spam-Status: No, score=-7.8 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gdb-testers@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-testers mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 May 2020 16:10:36 -0000 *** TEST RESULTS FOR COMMIT 41977d16e4ee5b9ad01abf2cfce6edbfb6d79541 *** commit 41977d16e4ee5b9ad01abf2cfce6edbfb6d79541 Author: Simon Marchi AuthorDate: Wed May 20 10:50:39 2020 -0400 Commit: Simon Marchi CommitDate: Wed May 20 10:50:50 2020 -0400 gdb/testsuite: check mmap ret val against MAP_FAILED Fixup a few spots in the testsuite that use mmap to consistently check the return value against MAP_FAILED. One spot in gdb.base/coredump-filter.c checked against NULL, that is wrong. The other spots either didn't check, or checked against -1. MAP_FAILED has the value -1, at least on Linux, but it's better to check against the documented define. gdb/testsuite/ChangeLog: PR gdb/26016 * gdb.base/coredump-filter.c (do_mmap): Check mmap ret val against MAP_FAILED. * gdb.base/coremaker.c (mmapdata): Likewise. * gdb.base/jit-reader-host.c (main): Likewise. * gdb.base/sym-file-loader.c (load): Likewise. (load_shlib): Likewise. diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 67522a7640..050a793ed0 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,13 @@ +2020-05-20 Simon Marchi + + PR gdb/26016 + * gdb.base/coredump-filter.c (do_mmap): Check mmap ret val + against MAP_FAILED. + * gdb.base/coremaker.c (mmapdata): Likewise. + * gdb.base/jit-reader-host.c (main): Likewise. + * gdb.base/sym-file-loader.c (load): Likewise. + (load_shlib): Likewise. + 2020-05-20 Tom Tromey * gdb.ada/array_char_idx.exp: Recognize initialized array. diff --git a/gdb/testsuite/gdb.base/coredump-filter.c b/gdb/testsuite/gdb.base/coredump-filter.c index f53a933a72..5786c6fe09 100644 --- a/gdb/testsuite/gdb.base/coredump-filter.c +++ b/gdb/testsuite/gdb.base/coredump-filter.c @@ -29,7 +29,7 @@ do_mmap (void *addr, size_t size, int prot, int flags, int fd, off_t offset) { void *ret = mmap (addr, size, prot, flags, fd, offset); - assert (ret != NULL); + assert (ret != MAP_FAILED); return ret; } diff --git a/gdb/testsuite/gdb.base/coremaker.c b/gdb/testsuite/gdb.base/coremaker.c index 55330fd3e8..3cc97e1e8e 100644 --- a/gdb/testsuite/gdb.base/coremaker.c +++ b/gdb/testsuite/gdb.base/coremaker.c @@ -77,7 +77,7 @@ mmapdata () /* Now map the file into our address space as buf2 */ buf2 = (char *) mmap (0, MAPSIZE, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); - if (buf2 == (char *) -1) + if (buf2 == (char *) MAP_FAILED) { perror ("mmap failed"); return; diff --git a/gdb/testsuite/gdb.base/jit-reader-host.c b/gdb/testsuite/gdb.base/jit-reader-host.c index f9c4833083..0cf653f1fb 100644 --- a/gdb/testsuite/gdb.base/jit-reader-host.c +++ b/gdb/testsuite/gdb.base/jit-reader-host.c @@ -15,6 +15,7 @@ You should have received a copy of the GNU General Public License along with this program. If not, see . */ +#include #include #include #include @@ -59,6 +60,8 @@ main (int argc, char **argv) MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); char *code_end = code; + assert (code != MAP_FAILED); + /* "JIT" function_stack_mangle. */ memcpy (code_end, jit_function_stack_mangle_code, sizeof (jit_function_stack_mangle_code)); diff --git a/gdb/testsuite/gdb.base/sym-file-loader.c b/gdb/testsuite/gdb.base/sym-file-loader.c index 5fcabec4e1..c8074b8489 100644 --- a/gdb/testsuite/gdb.base/sym-file-loader.c +++ b/gdb/testsuite/gdb.base/sym-file-loader.c @@ -20,6 +20,7 @@ #include #include #include +#include #include "sym-file-loader.h" @@ -112,6 +113,8 @@ load (uint8_t *addr, Elf_External_Phdr *phdr, struct segment *tail_seg) mapped_addr = (uint8_t *) mmap ((void *) GETADDR (phdr, p_vaddr), GET (phdr, p_memsz), perm, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0); + assert (mapped_addr != MAP_FAILED); + mapped_size = GET (phdr, p_memsz); from = (void *) (addr + GET (phdr, p_offset)); @@ -255,7 +258,7 @@ load_shlib (const char *file) } addr = (uint8_t *) mmap (NULL, fsize, PROT_READ, MAP_PRIVATE, fd, 0); - if (addr == (uint8_t *) -1) + if (addr == MAP_FAILED) { perror ("mmap failed."); return NULL;