From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 87BD748B3F for ; Tue, 18 Nov 2025 15:48:20 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 78E0C402E4; Tue, 18 Nov 2025 15:48:20 +0100 (CET) Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) by mails.dpdk.org (Postfix) with ESMTP id 9717C40265 for ; Tue, 18 Nov 2025 15:48:18 +0100 (CET) Received: by mail-wm1-f54.google.com with SMTP id 5b1f17b1804b1-477632d45c9so41469915e9.2 for ; Tue, 18 Nov 2025 06:48:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1763477298; x=1764082098; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=vahx2dbe9IL7hh0XBKleIPl4gb+Ct56cgeSZ59B8XD4=; b=GMXdKvY6hmJMMyToTF7QWrk9rtJzXMyYrkM6ub1ZxxENM4mcQLitN/MvAHBosvdmdz nqlvj1Qv3jC+UL9Xr3Gxh2tY/sWBknCPU2khwOjO5wJcyvjGx+DTrcNtD7WjLPRf/tDn nchrVMIGscarYGIMfotMTvjKZn3w8XIRQ0NgIr/DhdifdnXrr5IXkqUwzfsyF9xxH+Wl YSXXv0CkakaJN7nRj/XgqBQ2QFr3pNn+BrL5GOOhvkyK7vPjEFtGxEgsWY2XPZOa2Iu+ iA6pADoNOQFk+bfu82ewZwkuTKNaZz3pTYGpV/txn0g4Q7RRBldW0RFlzSf1ToXlDNjz tpfg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763477298; x=1764082098; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=vahx2dbe9IL7hh0XBKleIPl4gb+Ct56cgeSZ59B8XD4=; b=MrmZfsWqzMWATfWO5WdgBhHWMiHjQsGdwzHS1Lg8JrqYzlaDmytmWUavjPYu308twG z49p6ZDDQH1JI5uJV3Xvve7KZ7uRcL0Pz++CbN8q29Ao75LMN3VyXKJDVIBGryS0n0z/ H7p7X04iBAs96TvC9lYRiOiImPYEr6ipGCMzxB0yabP28yb8EAPcBBfRhAHSRWxb1z6S DZgYROckgjB79K53PFd3ctS1zjWaQ7NUPr7ZxxbT+dJ9HJ/gVwdQfZ7gbTsHMOly9S7i q9WCw1870MctrN53ALbsLwefHROq6jNgXrusaWuK+JTFq48YKJ2vAYvgkRkXFKjR/3+l sLNA== X-Forwarded-Encrypted: i=1; AJvYcCV8WTLObjLjjE7bJmRICAER1QftzOIlNJOtJRSaZncEBi51aczowVEKNaygQzEFcO/oXbbsrus=@dpdk.org X-Gm-Message-State: AOJu0Yym+dLPwGnXd0ikde91R128jfCXVY6NJFFz2gEQJpC4E6aURNQO 207BmI+rivv9Y828rK/GXzn035LQRCfc/GugiR6v5FeNsuUO0+Zb2F/efiHMPHahBQI= X-Gm-Gg: ASbGnctmRgJkZK6XpS2M98sQLZVRMomBDSlXo18S32qvNj6XFWCZjJQ8Ys1FZS7d4m2 RWI9R02b7mVbIC983HVuMJE4hf5kn/9f6+nF2XVhbeKXO0BJA15+X6duTye2xQHiHmlnKCgGQ+h JZQPx/2hM9LC4di5LMOQas5562GKNa49hTOaQ4wJJmu26OKRTZOyq33kZTFM8PbpbZU16nZGOhC 7n0r286jw4663HzMmMT7i2JqBL2oVZfGcowpaj0XR66cRnSdCxX8SnrONYZ3O/tHqBGdSN5TZlk B/Jzl7HgCDHeAkYK0ACJEWvXIgHzEvkyaxvKNkNyIQqxJ+K7vWXXPJkfNCAVYjyLoZQg/IRGuk2 qfAQ40ZwY9edIhpObT8ZPmyLGD5HWAh5DVDx80SSJ7GXF2+pCZN+v/67/CORQGrdIWx7/wl3Hee bd4+7PvjvbbmkllLOWQzv/Pvi8pLtd+cb7nLhQL/6aigqj0qsO3M4bvsleQrcyWb8= X-Google-Smtp-Source: AGHT+IF85X4nZnQt8Wo7Tv8T86jJLu9L9e2ezfkfPLd/X5bwU2XWGMHEB2lyvJ6+cxcLy9XRUwUELQ== X-Received: by 2002:a05:600c:470d:b0:477:8985:4036 with SMTP id 5b1f17b1804b1-4778fe41af3mr148253985e9.1.1763477298039; Tue, 18 Nov 2025 06:48:18 -0800 (PST) Received: from phoenix.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-477a9deeb96sm18515555e9.4.2025.11.18.06.48.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 18 Nov 2025 06:48:17 -0800 (PST) Date: Tue, 18 Nov 2025 06:48:12 -0800 From: Stephen Hemminger To: fengchengwen Cc: , , Cristian Dumitrescu , Bruce Richardson Subject: Re: [PATCH v3 5/6] test: fix impossible format-truncation in cfgfiles Message-ID: <20251118064812.19a5dffb@phoenix.local> In-Reply-To: <79049dc3-8c0b-41f2-bb02-73c18766377c@huawei.com> References: <20251115193409.99740-1-stephen@networkplumber.org> <20251117175332.14014-1-stephen@networkplumber.org> <20251117175332.14014-6-stephen@networkplumber.org> <79049dc3-8c0b-41f2-bb02-73c18766377c@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Tue, 18 Nov 2025 10:22:35 +0800 fengchengwen wrote: > On 11/18/2025 1:52 AM, Stephen Hemminger wrote: > > Although, it is not possible on Linux (which always uses /tmp) > > the compiler complains about possible snprintf() truncation. > > Simplest fix is to just increase the size of the filename variable. > > > > Fixes: be22019a58c4 ("test: restore cfgfile tests") > > Cc: stable@dpdk.org > > > > Signed-off-by: Stephen Hemminger > > --- > > app/test/test_cfgfile.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/app/test/test_cfgfile.c b/app/test/test_cfgfile.c > > index b189d9d7a5..823b48e1fa 100644 > > --- a/app/test/test_cfgfile.c > > +++ b/app/test/test_cfgfile.c > > @@ -148,7 +148,7 @@ static int > > test_cfgfile_realloc_sections(void) > > { > > struct rte_cfgfile *cfgfile; > > - char filename[PATH_MAX]; > > + char filename[PATH_MAX + NAME_MAX]; > > there are many other function invoke make_tmp_file(), such as test_cfgfile_invalid_section_header. > > I think we could modify make_tmp_file(): snprintf(filename, PATH_MAX - NAME_MAX, ...). > > > int ret; > > const char *value; > > > I think code can use existing make_tmp_file() function and avoid the problem of conflict on filename in /tmp.