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 7A3B0A0A02 for ; Mon, 17 May 2021 18:11:03 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 72F0541101; Mon, 17 May 2021 18:11:03 +0200 (CEST) Received: from youngberry.canonical.com (youngberry.canonical.com [91.189.89.112]) by mails.dpdk.org (Postfix) with ESMTP id 23C52410EA for ; Mon, 17 May 2021 18:11:01 +0200 (CEST) Received: from 2.general.paelzer.uk.vpn ([10.172.196.173] helo=Keschdeichel.fritz.box) by youngberry.canonical.com with esmtpsa (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.93) (envelope-from ) id 1lifpa-0007X4-AX; Mon, 17 May 2021 16:10:58 +0000 From: Christian Ehrhardt To: Anatoly Burakov Cc: Andrew Boyer , David Marchand , dpdk stable Date: Mon, 17 May 2021 18:07:15 +0200 Message-Id: <20210517161039.3132619-6-christian.ehrhardt@canonical.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210517161039.3132619-1-christian.ehrhardt@canonical.com> References: <20210517161039.3132619-1-christian.ehrhardt@canonical.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'fbarray: fix log message on truncation error' has been queued to stable release 19.11.9 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 Sender: "stable" Hi, FYI, your patch has been queued to stable release 19.11.9 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 05/19/21. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/cpaelzer/dpdk-stable-queue This queued commit can be viewed at: https://github.com/cpaelzer/dpdk-stable-queue/commit/87ec379c3e9e73709d3e15d667836d82faf7f863 Thanks. Christian Ehrhardt --- >From 87ec379c3e9e73709d3e15d667836d82faf7f863 Mon Sep 17 00:00:00 2001 From: Anatoly Burakov Date: Fri, 19 Feb 2021 17:54:45 +0000 Subject: [PATCH] fbarray: fix log message on truncation error [ upstream commit 2a2ebeab9e3309ab332baefdbcc1caa7e62ecc82 ] When file truncation fails, the log message attempts to print a path of file we failed to truncate, but this path was never set to anything and, what's worse, was uninitialized. Fix it by passing path from the caller. Coverity issue: 366122 Fixes: c44d09811b40 ("eal: add shared indexed file-backed array") Reported-by: Andrew Boyer Signed-off-by: Anatoly Burakov Reviewed-by: David Marchand --- lib/librte_eal/common/eal_common_fbarray.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/librte_eal/common/eal_common_fbarray.c b/lib/librte_eal/common/eal_common_fbarray.c index de7e772042..c45c1189b3 100644 --- a/lib/librte_eal/common/eal_common_fbarray.c +++ b/lib/librte_eal/common/eal_common_fbarray.c @@ -80,9 +80,8 @@ get_used_mask(void *data, unsigned int elt_sz, unsigned int len) } static int -resize_and_map(int fd, void *addr, size_t len) +resize_and_map(int fd, const char *path, void *addr, size_t len) { - char path[PATH_MAX]; void *map_addr; if (ftruncate(fd, len)) { @@ -794,7 +793,7 @@ rte_fbarray_init(struct rte_fbarray *arr, const char *name, unsigned int len, goto fail; } - if (resize_and_map(fd, data, mmap_len)) + if (resize_and_map(fd, path, data, mmap_len)) goto fail; } ma->addr = data; @@ -900,7 +899,7 @@ rte_fbarray_attach(struct rte_fbarray *arr) goto fail; } - if (resize_and_map(fd, data, mmap_len)) + if (resize_and_map(fd, path, data, mmap_len)) goto fail; /* store our new memory area */ -- 2.31.1 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2021-05-17 17:40:29.825261485 +0200 +++ 0006-fbarray-fix-log-message-on-truncation-error.patch 2021-05-17 17:40:29.091808846 +0200 @@ -1 +1 @@ -From 2a2ebeab9e3309ab332baefdbcc1caa7e62ecc82 Mon Sep 17 00:00:00 2001 +From 87ec379c3e9e73709d3e15d667836d82faf7f863 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 2a2ebeab9e3309ab332baefdbcc1caa7e62ecc82 ] + @@ -12 +13,0 @@ -Cc: stable@dpdk.org @@ -22 +23 @@ -index d974f3dab7..592ec58594 100644 +index de7e772042..c45c1189b3 100644 @@ -25 +26 @@ -@@ -81,9 +81,8 @@ get_used_mask(void *data, unsigned int elt_sz, unsigned int len) +@@ -80,9 +80,8 @@ get_used_mask(void *data, unsigned int elt_sz, unsigned int len) @@ -35,3 +36,2 @@ - if (eal_file_truncate(fd, len)) { -@@ -792,7 +791,7 @@ rte_fbarray_init(struct rte_fbarray *arr, const char *name, unsigned int len, - if (eal_file_lock(fd, EAL_FLOCK_SHARED, EAL_FLOCK_RETURN)) + if (ftruncate(fd, len)) { +@@ -794,7 +793,7 @@ rte_fbarray_init(struct rte_fbarray *arr, const char *name, unsigned int len, @@ -38,0 +39 @@ + } @@ -45,2 +46 @@ -@@ -895,7 +894,7 @@ rte_fbarray_attach(struct rte_fbarray *arr) - if (eal_file_lock(fd, EAL_FLOCK_SHARED, EAL_FLOCK_RETURN)) +@@ -900,7 +899,7 @@ rte_fbarray_attach(struct rte_fbarray *arr) @@ -47,0 +48 @@ + }