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 51FB245CFC; Thu, 14 Nov 2024 03:37:51 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CFB6D4161A; Thu, 14 Nov 2024 03:37:50 +0100 (CET) Received: from mail-pg1-f178.google.com (mail-pg1-f178.google.com [209.85.215.178]) by mails.dpdk.org (Postfix) with ESMTP id F377840268 for ; Thu, 14 Nov 2024 03:37:49 +0100 (CET) Received: by mail-pg1-f178.google.com with SMTP id 41be03b00d2f7-7f3e30a43f1so58801a12.1 for ; Wed, 13 Nov 2024 18:37:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1731551869; x=1732156669; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=P0Vjv5Tads0XWWauGK6DOHQVj0nPL9JrKq3Zfhj3vJ8=; b=u/Y6OU6VL27LJyFUtXb40QojL6m4zHX9/pRryAlVfukcXcdKr8k5+qPqhVPk6CEJNL mu0jkaUKZwdSd0JyY8Q2Z7pS9nlG1EJtjl6Zfa3PbGdZudiDn2YRoX1G+uSS5CdTFeLS sFplHebE4TSBMeUxKiACcP2HGuvbGMrvyz2GmfHFiatTr9TUPm7loCFnvP21vyrBGtqk NGfaVcp5wSrcVVUREfsllxr9htskaka/gceOcRbU8qGiVPhB3ALmZRYTUjd3G+KLxi2/ CboBKX+/tcHy4BD/PvxprtSbwtWv8o4+mXZJRQP/Y7eL4A2bKxwqK4xDstNzMiNOdGbA dE3Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1731551869; x=1732156669; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=P0Vjv5Tads0XWWauGK6DOHQVj0nPL9JrKq3Zfhj3vJ8=; b=xF954COSrZmVmu/Iemii1xfoiFLTOeyeJafruyPFZMSkCClm4SSIRLAK6PWZdejegq HqW8xBSrpBssGvmgAJXZ5hiNZJSU179p56GQM9rrgYS7ydfbFYk5FrrLtIPcJ7XnJRRP WEHa7BUnjK/zogbhZ5g6Wi+9KHiM/uzMGH2x1MJ4d42orVX7sm/p4flD5sd4++mXKQKm qYL/TI+qHrflodyCKtEYZnb4xJbfV8RJiGGL30mrWtdxeq6fSIqflE7dkKHa+MDVshPw qpZXAZLi6jo4hkdnwtZEfWs4WhdEBt8bqYu5UK42bR7IOxjwruzesspB0eaGxWr8jd58 hZyw== X-Gm-Message-State: AOJu0Yxm27DuOqMsPMBlgy4EFw9Dc2xGlyaXOvkcC5Clfd2SoLRMBqCZ sJTY9gVpSq70J1I3hsLTuXNHilTr2elAIvB9qV96h1cRgyZeJyK0z1ul+UMR4hjeqrBrXhod84H j X-Google-Smtp-Source: AGHT+IGbhwVKaFlu4KrMm0s4gn5yzlQYWNQbPF++GWCsN4ASHMPXNNYYxeKc7RuTea7j90dbIrrtSA== X-Received: by 2002:a05:6a20:9183:b0:1db:e338:ab0a with SMTP id adf61e73a8af0-1dc8336123dmr991777637.8.1731551868875; Wed, 13 Nov 2024 18:37:48 -0800 (PST) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-7246a5d2bb5sm150976b3a.29.2024.11.13.18.37.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Nov 2024 18:37:48 -0800 (PST) From: Stephen Hemminger To: dev@dpdk.org Cc: Stephen Hemminger Subject: [PATCH v3 00/11] memset security handling Date: Wed, 13 Nov 2024 18:35:49 -0800 Message-ID: <20241114023738.141821-1-stephen@networkplumber.org> X-Mailer: git-send-email 2.45.2 In-Reply-To: <20241114011129.451243-1-stephen@networkplumber.org> References: <20241114011129.451243-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org This series handles memset related bugs indentified by PVS Studio. The root cause is that Gcc and other compilers are free to optimize away memset called before free. Most of the places memset was being used like this were bogus; probably some developer debug habit, and can be safely removed. v3 - fix spelling - fix issues reported in Napatech NIC Stephen Hemminger (11): eal: introduce new secure memory fill eal: add new secure free function crypto/qat: force zero of keys crypto/qat: fix size calculation for memset crypto/qat: use secure memset bus/uacce: remove memset before free compress/octeontx: remove unnecessary memset test: remove unneeded memset net/ntnic: remove unnecessary void cast net/ntnic: check result of malloc net/ntnic: remove unnecessary memset app/test/test_cmdline_cirbuf.c | 2 -- drivers/bus/uacce/uacce.c | 1 - drivers/compress/octeontx/otx_zip.c | 1 - drivers/compress/octeontx/otx_zip_pmd.c | 2 -- drivers/crypto/qat/qat_asym.c | 5 +--- drivers/crypto/qat/qat_sym_session.c | 27 +++++++++-------- drivers/net/ntnic/nim/i2c_nim.c | 2 +- drivers/net/ntnic/nthw/core/nthw_hif.c | 5 +--- drivers/net/ntnic/nthw/core/nthw_iic.c | 5 +--- drivers/net/ntnic/nthw/core/nthw_pcie3.c | 5 +--- drivers/net/ntnic/nthw/core/nthw_rpf.c | 5 +--- drivers/net/ntnic/nthw/core/nthw_sdc.c | 5 +--- drivers/net/ntnic/nthw/core/nthw_si5340.c | 5 +--- .../ntnic/nthw/flow_filter/flow_nthw_cat.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_csu.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_flm.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_hfu.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_hsh.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_ifr.c | 2 +- .../ntnic/nthw/flow_filter/flow_nthw_info.c | 7 ++--- .../net/ntnic/nthw/flow_filter/flow_nthw_km.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_pdb.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_qsl.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_rpp_lr.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_slc_lr.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_tx_cpy.c | 3 +- .../ntnic/nthw/flow_filter/flow_nthw_tx_ins.c | 7 ++--- .../ntnic/nthw/flow_filter/flow_nthw_tx_rpl.c | 7 ++--- drivers/net/ntnic/nthw/nthw_rac.c | 4 ++- drivers/net/ntnic/ntnic_ethdev.c | 2 +- lib/eal/common/rte_malloc.c | 30 +++++++++++++++---- lib/eal/include/rte_malloc.h | 18 +++++++++++ lib/eal/include/rte_string_fns.h | 27 +++++++++++++++++ lib/eal/version.map | 3 ++ 34 files changed, 126 insertions(+), 124 deletions(-) -- 2.45.2