From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailsec113.isp.belgacom.be (mailsec113.isp.belgacom.be [195.238.20.109]) by sourceware.org (Postfix) with ESMTPS id 9E6C33858D20 for ; Sun, 20 Nov 2022 17:30:35 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 9E6C33858D20 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=skynet.be Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=skynet.be DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=skynet.be; i=@skynet.be; q=dns/txt; s=rmail; t=1668965436; x=1700501436; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=dKjn3O0sEBw4VYUsYS4Ykrw6LNHuScT0CgAHwc+/ecI=; b=ROZB8QhAi0uF7SLF/9JccI0McJ8h2MAxKsT9RJ2RO9jxIpkTgi/vFNOY nojc1kLIZLKqYij29P4TsSG+vGXQHGl/yvxQ9yZgLJ6D1uq6cLVliFh0n 7QIMH9HZVqPrbwsr5EeOCbqpv/7z7msyUOKzbZGpGIOpx9ngaH1t/eUYd g=; X-ExtLoop: 1 X-IPAS-Result: =?us-ascii?q?A2DWBAC6Ynpj/1uGgG1agRKcN58yDwEBAQEBAQEBAQlEB?= =?us-ascii?q?AEBhQWFBCY4EwECBAEBAQEDAgMBAQEBAQEDAQEGAQEBAQEBBgQBgRuFL0aCN?= =?us-ascii?q?SKDfysLAUaEToMjsz4zgQGEcJpmgWeBQItOhXmBVUSED4R5gQaFcQSYMAMJA?= =?us-ascii?q?wcFSUADCxgNFjIKEzcbWA4JHxwlDQUGEgMgbAUHOg8oL2crHBsHgQwqKBUDB?= =?us-ascii?q?AQDAgYTAyICDSkxFAQpEw0rJ28JAgMhagMDBCgsAwkhHwcnJjwHVjoFAwIPI?= =?us-ascii?q?DgGAwkDAiJVdC8SFAUDCxUlCAVLBAg5BQZTEgIKEQMSDyxFDkg+ORYGJ0IBM?= =?us-ascii?q?Q4OFANeSx2BAQSCJgqcBQc9FigTgT00CGWSZAeSGJ1GNAeDa4FHBgyeYRoyq?= =?us-ascii?q?RUBlzSjJ4RRgXmBfm2DO1EZD45XjhqBLwIHCwEBAwmICYJZAQE?= IronPort-PHdr: A9a23:NrodlhJcpMvevPLTU9mcuPJmWUAX0o4c3iYr45Yqw4hDbr6kt8y7e hCFubMw0BSWA82bs6sC17CN9fi4GCQp2tWoiDg6aptCVhsI2409vjcLJ4q7M3D9N+PgdCcgH c5PBxdP9nC/NlVJSo6lPwWB6nK94iQPFRrhKAF7Ovr6GpLIj8Swyuu+54Dfbx9HiTajbr5+N hW7oAreusQUgIZpN6I9xgfUrndSdOla221lKUiPkxrg48u74YJu/TlXt/897cBLTL/0f74/T bxWDTQmN3466cj2vhTdTgWB+2URXHwOnhVHHwbK4hf6XozssiThrepyxDOaPcztQr8qXzmp8 rpmRwXpiCcDMD457X3Xh8lth69VvB6tuxpyyJPSbYqINvRxY7ndcMsaS2RfQMtfSiJPDIC7Y YQAAOQMJvpUoornqlcStxayGRWgCeXywTFInH/22qg63vw7HwHG3QwgG9MOsGzMrNrrKawdU fq6zLPPzTXacvNW2Cny6JLTchs8pvyMRbJwccvVyUkuDwPFlkufpZbrPjyPzOQNtHGb7/dhV e2xkW4nsBp8oiOsxsYsjInGmJ4Vxkrf+ipn2Ys4I8CzR0Fnb9C+CpRQqz2aOJVsQsMkW2xlt js3x7wJtJKmfCYEyJQpyhrcZvCaboWE/BztWumPLDl2mH5oeryyiguz/EWvxOPyWde53EhLo yZYndTBqnEA2hrO4caEUvtw5lqt1SqL2gzJ9+1IP0M5mbDGJ5MvzbM8jJ4evVnFEyTrgkv5l rWWeV8h+uWw7uTnZajpqYGEOo9vjwH+LrwumsuiAeQkKgQOX3aU+eC71LD7+E32Xa9Gjv0xk qncrp/WPcUbpqinDA9Jyosv9gqzAjO83NgFg3UKL0hJdA+JgoXmIV3DJO30Ae+6g1u2kTdrw /7GPqfmApXINnXDiLfhfbd5605d0Aoz1c5Q64haC74bOvLzXVbxtMHZDhAnKQy02P3qCM5+1 oIeX2KAHLOZPLnJvlCW/u4vJfKDa5cPuDnhM/gl++LujXghlFMDZaWpx4cYaGikHvR6JEWUe XXsjcoaHmsTpgoxVvDqh0GCUTFNfHa/RLk85jYmCIK8EYjMWI6sj6ab3CilBJFWYXpGCl+UH Xfya4qEQ+sMaD6VIsJ5ijwEVbmhS4sg1RG2rA/11aBrL+TO9S0CspLjzcV15+zNmhEo7jx5F N+d02KNTmFygGwIWyU607thrh819lDW3qRyxuRRCddT6uhhSQAnL5XR0OVgBpb1QA2SUM2OT QOeQtSiACkpQ5oOysUJelt8Fs+5xkTb3yuuAqcNmvqUDYYz67/d0mLqD91+2nDLyO8rgg91E YN0KWS6i/snpEDoDInTnhDB/5s= IronPort-Data: A9a23:QKkv+aC2aZdETxVW/0Tiw5YqxClBgxIJ4kV8jS/XYbTApDIr0zZVm jYbUGjTbKyNazDxc4xzPYS18UxS65OAy4JgOVdlrnsFo1CmCSbm6XV1Cm+qYkt+++WaFBoPA /02M4WGdoZtJpPljk/FGqD7qnVh3r2/SLP5CerVUgh8XgYMpB0J0HqPoMZkxN826TSFK1nV4 4mr+peHYAbNNwNcawr41YrS8HuDg9yv4Fv0jnRmDdhXsVnXkWUiDZ53Dcld+FOhH+G4tsbjL wry5OnRElHxpn/BOfv5+lrPSXDmd5aJVeS4ZtW6bID56vRKjnRaPq/Wr5PwY28P49mCt4gZJ NmgKfVcRC9xVpAgltjxXDFCGC51M7N/xobDeyikldKR/XOBTFrVlqAG4EEeZeX0+85yDSdO8 vkVQNwPRknb1qTvmuL9E7IwwJV6RCXoFNp3VnVI1THYCfc+WZ2FXKzQ4sZF3ToqnehVHufYa tZfYzcHgBHoOkcWagZLVcJi9AuurkDUVXpKqnbJn7MMyXT4xxMywpHuFvOAL7RmQu0QxC50v Fnu8GjzRwkTKNefxCGt6XWxnOTCgirhVcQVDrLQ3uZqgVmS3nQeTgIfT1yivPi0kFWWQNFOL UEIvCAjxZXe72T6F5+kBEH9+STU+0dECpxZH/Y+rQ2czOzV+B7fDGUAQTdKb9lgvsJeqSEW6 2JlVujBXVRH2IB5g1rEnltIhVte8hT54YPPieHogOfFDxTeTFkPsy/y IronPort-HdrOrdr: A9a23:Vupy3Khl2Ug3+omb2GbXcCEqlHBQXi0ji2hC6mlwRA09TyX4rb HMoB1/73TJYVkqN03I9ergBED4ewK7yXct2/h3AV7AZmnbUQmTQL2KhLGKq1eMJ8SUzIBgPM lbAtNDIey1IV9mjdvrpCmUeuxQuuWvweSNrcf65VFEZyACUdAE0y5JTj++Vml7WQFKDYcwUI CR4ccCuyahdB0sH6aG76A+LpH+Tgvw5erbSBoPBxss7gGFjHez5Ln2VwSF3hp2aUIq/Ysf X-IronPort-Anti-Spam-Filtered: true X-ProximusIPWarmup: true Received: from 91.134-128-109.adsl-dyn.isp.belgacom.be (HELO md.home) ([109.128.134.91]) by relay.proximus.be with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Nov 2022 18:30:34 +0100 From: Philippe Waroquiers To: gdb-patches@sourceware.org Cc: Philippe Waroquiers Subject: [RFA] When getting the locno of a bpstat, handle the case of bp with null locations. Date: Sun, 20 Nov 2022 18:30:24 +0100 Message-Id: <20221120173024.3647464-1-philippe.waroquiers@skynet.be> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-12.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: The test py-objfile.exp unloads the current file while debugging the process. This results in bpstat bs->b->loc to become nullptr. Handle this case in breakpoint.c:bpstat_locno. Note: GDB crashes on this problem with an internal error, but the end of gdb summary shows: ... === gdb Summary === # of expected passes 36 The output also does not contain a 'FAIL:'. After the dix, the nr of expeted passes increased. In the gdb.log output, one can see: ... Fatal signal: Segmentation fault ----- Backtrace ----- 0x55698905c5b9 gdb_internal_backtrace_1 ../../binutils-gdb/gdb/bt-utils.c:122 0x55698905c5b9 _Z22gdb_internal_backtracev ... ERROR: Couldn't send python print(objfile.filename) to GDB. ERROR: : spawn id exp9 not open while executing "expect { -i exp9 -timeout 10 -re ".*A problem internal to GDB has been detected" { fail "$message (GDB internal error)" gdb_internal_error..." ("uplevel" body line 1) invoked from within .... Wondering if it might be possible to improve gdb_test to have gdb_test "python print(objfile.filename)" "None" \ "objfile.filename after objfile is unloaded" reporting a failed result instead of just producing the internal error. --- gdb/breakpoint.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 5b691673a0e..a161b78a8aa 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -4486,7 +4486,7 @@ bpstat_locno (const bpstat *bs) int locno = 0; - if (b != nullptr && b->loc->next != nullptr) + if (b != nullptr && b->loc != nullptr && b->loc->next != nullptr) { const bp_location *bl_i; -- 2.30.2