From nobody Tue Nov 26 19:56:24 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1705787206; cv=none; d=zohomail.com; s=zohoarc; b=K4diODDDUwAppfeegC+zjeWKdaPUT1FlT2IGVDLVwfnL3I40heQY70pXKyuf3Ax6RrLufLS+r+Sp6gxEHrE5WEskREQklt0nnTUcIOsYsB5lqQ1TYBcQWYnTlCfgzDG+O0bdwkNIsD3hGygaAyeDageENwT23vqUqkzDTuDcyNQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1705787206; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=gKdhzUN3vk16j953HmL0OBTJP0NJa3aSCwDkuQRo1hg=; b=e6ufbYyNIOa7o9QnbGgpHeEASV9sHLTOZqMotyOj0Zqr3RuqK+mK266L7JUGF81jHSpr0l+TfrI4mvTRJdo80cADV4jJPAqrf3RESQj9pUmTcapGLJVMJEV2LcK5Pqd9Azhfi9mI5C34Y8ac7oreADtgD4750o6zvfT01hXw4fU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1705787205875919.8468926393763; Sat, 20 Jan 2024 13:46:45 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rRJA1-0006tl-8O; Sat, 20 Jan 2024 16:45:53 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rRJ9x-0006s6-Ov for qemu-devel@nongnu.org; Sat, 20 Jan 2024 16:45:49 -0500 Received: from todd.t-8ch.de ([159.69.126.157]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rRJ9u-0008A6-QM for qemu-devel@nongnu.org; Sat, 20 Jan 2024 16:45:49 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=t-8ch.de; s=mail; t=1705787141; bh=GKphi3SnzBrLWdNbMAbEC03yt732UAoayHsOeiLhWAg=; h=From:Date:Subject:References:In-Reply-To:To:Cc:From; b=nw4w/gon05X/JaM8EicXOtYB0oqvrJHe1XHvwTvJ8OyifFiLGFndyMbEmS7hYvmTf Tipo8fMDVKEkbWr4n4benw74BTNAwd3Z3KhMKQXjMdTcRiMaryvmx+fmi7P/wSt696 I3Syzj4wY9OCoq3zbhRry54L+G5u9qCWGiXx9S0E= From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= Date: Sat, 20 Jan 2024 22:45:24 +0100 Subject: [PATCH v3 1/2] linux-user/elfload: test return value of getrlimit MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240120-qemu-user-dumpable-v3-1-6aa410c933f1@t-8ch.de> References: <20240120-qemu-user-dumpable-v3-0-6aa410c933f1@t-8ch.de> In-Reply-To: <20240120-qemu-user-dumpable-v3-0-6aa410c933f1@t-8ch.de> To: Laurent Vivier Cc: qemu-devel@nongnu.org, =?utf-8?q?Thomas_Wei=C3=9Fschuh?= , Richard Henderson X-Mailer: b4 0.12.4 X-Developer-Signature: v=1; a=ed25519-sha256; t=1705787141; l=891; i=thomas@t-8ch.de; s=20221212; h=from:subject:message-id; bh=GKphi3SnzBrLWdNbMAbEC03yt732UAoayHsOeiLhWAg=; b=tWc9LNQQ59NCpb+pKekzt6s8W6APkl8pL6nsOI2uC3iFrnhlbWKvxkb9k5Mj6tNctG+50sK1Z 6oe2JtloolxBy9kExuL0dX+qybJTCD8o3vdsYoSGJ5pK/ODY8o31x95 X-Developer-Key: i=thomas@t-8ch.de; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=159.69.126.157; envelope-from=thomas@t-8ch.de; helo=todd.t-8ch.de X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @t-8ch.de) X-ZM-MESSAGEID: 1705787207992100007 Should getrlimit() fail the value of dumpsize.rlimit_cur may not be initialized. Avoid reading garbage data by checking the return value of getrlimit. Reviewed-by: Richard Henderson Signed-off-by: Thomas Wei=C3=9Fschuh Reviewed-by: Philippe Mathieu-Daud=C3=A9 --- linux-user/elfload.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index cf9e74468b11..c5968719380a 100644 --- a/linux-user/elfload.c +++ b/linux-user/elfload.c @@ -4667,9 +4667,9 @@ static int elf_core_dump(int signr, const CPUArchStat= e *env) init_note_info(&info); =20 errno =3D 0; - getrlimit(RLIMIT_CORE, &dumpsize); - if (dumpsize.rlim_cur =3D=3D 0) + if (getrlimit(RLIMIT_CORE, &dumpsize) =3D=3D 0 && dumpsize.rlim_cur = =3D=3D 0) { return 0; + } =20 corefile =3D core_dump_filename(ts); =20 --=20 2.43.0