From patchwork Thu Feb 29 15:35:41 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Aktemur, Tankut Baris" X-Patchwork-Id: 86595 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id E8F2E3858409 for ; Thu, 29 Feb 2024 15:36:49 +0000 (GMT) X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.8]) by sourceware.org (Postfix) with ESMTPS id A36D63858C74 for ; Thu, 29 Feb 2024 15:36:18 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A36D63858C74 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=intel.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org A36D63858C74 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=192.198.163.8 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1709220981; cv=none; b=rGP7hzAmVWAiv0ebdEbqgYvEc6FgKpFU6/lDcxPGc9cTE/IZkvUAEcaXxenW1tenV7NHyJVXYAnp054XiHW5jFWVijy9F9/rua06CSLmnW/pL2YeOIZkMo1qdVyvkA3yZX59om/kTlLQS1/jTQTYUVO5/TtSg0yHd7n3HIJiCHo= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1709220981; c=relaxed/simple; bh=uYLf3dX4pi5uNA+qYJ2P0Omlb7vx56u5L8tKicXMrXk=; h=DKIM-Signature:From:To:Subject:Date:Message-Id:MIME-Version; b=ZrQUopbY/4zVG4s5geHeFTKBbkfanSAIpd5QPjiljVw8V2Dmr+bm1SYA8AGsk3uq2rCwz0F2XQYOAVhArPdS+EapOwD5pwnOaOtu5smLpyQL6W8ngVeLjWTKZZ3XAo8Vr8ZCgQMN4kZ6yNTwkmLHdjpsm9RWOgtV5nTth7qbDK8= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1709220979; x=1740756979; h=from:to:subject:date:message-id:mime-version: content-transfer-encoding; bh=uYLf3dX4pi5uNA+qYJ2P0Omlb7vx56u5L8tKicXMrXk=; b=Vz2esLydcvI1mY5yBXk8CyrfteksTrRF9xb65xruym+u8tHqkNKncAqK bKYNgMMhblb459tfuGoqqVPGInA2gyW/24c4cSOt5wxGeJFS5XnBETx3m iWMskdZgcRFwHbwfADxPKxNL4YHO8A3CmV67AZXW7EO5bJKUuTaJrTW1s rhUvFLN+qjdjmW89t3I0D5Epr9CrG90sPRZm5KEKfyCtqgqsVUEFXGCOI NoUqq+KlrtKJt1txJJl5ei5z5ZiMjGDEVhM5yJziK6Xd2U2amQbUozdEA d605ja7ugpTva4Dz4940iUagKe8HD2bAYAOFyBytx51gLwZ79AyOAsCQL A==; X-IronPort-AV: E=McAfee;i="6600,9927,10999"; a="21241285" X-IronPort-AV: E=Sophos;i="6.06,194,1705392000"; d="scan'208";a="21241285" Received: from fmviesa002.fm.intel.com ([10.60.135.142]) by fmvoesa102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Feb 2024 07:35:56 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.06,194,1705392000"; d="scan'208";a="31048224" Received: from gkldtt-dev-004.igk.intel.com (HELO localhost) ([10.123.221.202]) by fmviesa002-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Feb 2024 07:35:56 -0800 From: Tankut Baris Aktemur To: gdb-patches@sourceware.org Subject: [PATCH] gdbserver: require_running_or_break for the 'z' packet Date: Thu, 29 Feb 2024 16:35:41 +0100 Message-Id: <20240229153541.3057573-1-tankut.baris.aktemur@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-Spam-Status: No, score=-9.4 required=5.0 tests=AC_FROM_MANY_DOTS, BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org Similar to several other packages that access/modify process state, the 'z' and 'Z' packets need to access the process. Hence, add a `require_running_or_break` as a pre-check. --- gdbserver/server.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/gdbserver/server.cc b/gdbserver/server.cc index 74c7763d777..8b9c4df50ae 100644 --- a/gdbserver/server.cc +++ b/gdbserver/server.cc @@ -4731,6 +4731,7 @@ process_serial_event (void) /* Fallthrough. */ case 'z': /* remove_ ... */ { + require_running_or_break (cs.own_buf); char *dataptr; ULONGEST addr; int kind;