From nobody Wed Apr 24 08:39:38 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail(p=none dis=none) header.from=antioche.eu.org ARC-Seal: i=1; a=rsa-sha256; t=1610475578; cv=none; d=zohomail.com; s=zohoarc; b=IBysNZr2lzgxefzn9N0wz6SVT3oWq8rqXyqE++QlSsGIR3neFHFTK8NSwUcfZZMjXGv+bT6vkltfKvZmTQXCVx1MSooIx/6GbqQBwMInKTWfh2UJjFMnbxEZydMErxoINPXFO+JOkVvXqHwUt8mke+pHPMehzNqk+TQy0v7yV9Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1610475578; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=S6XBHU5sXr4TbyK88Nue4vhMO2ra4ALa3T63hmkobfA=; b=e82dPAfKOCddw8/+8mPtfOgKNaZ+pUjJM8DPMn2jMtlgKdXrX8N5Q6Ge2+T1GcY+6zNYQy6PSCapgIyvby3a9sgAioeDyEiYx8+sXoLqgXKW82oBfn3dwgL6MzO+yVMJudf69kAIMFb1WLBojF5m6ToBGcFrOn3iNA5y+NWsn60= ARC-Authentication-Results: i=1; mx.zohomail.com; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1610475578065843.1543368482561; Tue, 12 Jan 2021 10:19:38 -0800 (PST) Received: from list by lists.xenproject.org with outflank-mailman.65879.116845 (Exim 4.92) (envelope-from ) id 1kzOGM-0005Hn-Ct; Tue, 12 Jan 2021 18:19:26 +0000 Received: by outflank-mailman (output) from mailman id 65879.116845; Tue, 12 Jan 2021 18:19:26 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1kzOGL-0005H7-TK; Tue, 12 Jan 2021 18:19:25 +0000 Received: by outflank-mailman (input) for mailman id 65879; Tue, 12 Jan 2021 18:19:24 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1kzOCU-0003Yo-1F for xen-devel@lists.xenproject.org; Tue, 12 Jan 2021 18:15:26 +0000 Received: from isis.lip6.fr (unknown [2001:660:3302:283c::2]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id fdf72f1a-0319-4c0d-bea6-1d6a95af51c8; Tue, 12 Jan 2021 18:13:59 +0000 (UTC) Received: from asim.lip6.fr (asim.lip6.fr [132.227.86.2]) by isis.lip6.fr (8.15.2/8.15.2) with ESMTP id 10CIDw4Z001664; Tue, 12 Jan 2021 19:13:58 +0100 (CET) Received: from armandeche.soc.lip6.fr (armandeche [132.227.63.133]) by asim.lip6.fr (8.15.2/8.14.4) with ESMTP id 10CIDupY004943; Tue, 12 Jan 2021 19:13:56 +0100 (MET) Received: by armandeche.soc.lip6.fr (Postfix, from userid 20331) id A3A897218; Tue, 12 Jan 2021 19:13:56 +0100 (MET) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: fdf72f1a-0319-4c0d-bea6-1d6a95af51c8 From: Manuel Bouyer To: xen-devel@lists.xenproject.org Cc: Manuel Bouyer , Ian Jackson , Wei Liu Subject: [PATCH] xenpmd.c: use dynamic allocation Date: Tue, 12 Jan 2021 19:12:41 +0100 Message-Id: <20210112181242.1570-21-bouyer@antioche.eu.org> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20210112181242.1570-1-bouyer@antioche.eu.org> References: <20210112181242.1570-1-bouyer@antioche.eu.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3 (isis.lip6.fr [132.227.60.2]); Tue, 12 Jan 2021 19:13:58 +0100 (CET) X-Scanned-By: MIMEDefang 2.78 on 132.227.60.2 Content-Type: text/plain; charset="utf-8" From: Manuel Bouyer On NetBSD, d_name is larger than 256, so file_name[284] may not be large enough (and gcc emits a format-truncation error). Use asprintf() instead of snprintf() on a static on-stack buffer. Signed-off-by: Manuel Bouyer Reviewed-by: Ian Jackson --- tools/xenpmd/xenpmd.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/xenpmd/xenpmd.c b/tools/xenpmd/xenpmd.c index 12b82cf43e..e432aad856 100644 --- a/tools/xenpmd/xenpmd.c +++ b/tools/xenpmd/xenpmd.c @@ -101,7 +101,7 @@ FILE *get_next_battery_file(DIR *battery_dir, { FILE *file =3D 0; struct dirent *dir_entries; - char file_name[284]; + char *file_name; int ret; =20 do=20 @@ -112,16 +112,16 @@ FILE *get_next_battery_file(DIR *battery_dir, if ( strlen(dir_entries->d_name) < 4 ) continue; if ( battery_info_type =3D=3D BIF )=20 - ret =3D snprintf(file_name, sizeof(file_name), BATTERY_INFO_FI= LE_PATH, + ret =3D asprintf(&file_name, BATTERY_INFO_FILE_PATH, dir_entries->d_name); else=20 - ret =3D snprintf(file_name, sizeof(file_name), BATTERY_STATE_F= ILE_PATH, + ret =3D asprintf(&file_name, BATTERY_STATE_FILE_PATH, dir_entries->d_name); /* This should not happen but is needed to pass gcc checks */ if (ret < 0) continue; - file_name[sizeof(file_name) - 1] =3D '\0'; file =3D fopen(file_name, "r"); + free(file_name); } while ( !file ); =20 return file; --=20 2.29.2