netdev - [PATCH net-next v2 0/2] bnxt_en: address string truncation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <20240813-bnxt-str-v2-0-872050a157e7@kernel.org>
Date: Tue, 13 Aug 2024 15:32:54 +0100
From: Simon Horman <horms@...nel.org>
To: Michael Chan <michael.chan@...adcom.com>
Cc: "David S. Miller" <davem@...emloft.net>, 
 Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, 
 Paolo Abeni <pabeni@...hat.com>, Pavan Chebbi <pavan.chebbi@...adcom.com>, 
 Przemek Kitszel <przemyslaw.kitszel@...el.com>, netdev@...r.kernel.org
Subject: [PATCH net-next v2 0/2] bnxt_en: address string truncation

Hi,

This series addresses several string truncation issues that are flagged
by gcc-14. I do not have any reason to believe these are bugs, so I am
targeting this at net-next and have not provided Fixes tags.

---
Changes in v2:
- Added patch:
  + bnxt_en: Extend maximum length of version string by 1 byte
- Dropped the following patches.
  - bnxt_en: check for irq name truncation
  - bnxt_en: check for fw_ver_str truncation
  + The approach I had taken was to return error on truncation, but the
    feedback I received was that it would be better to replace the last
    three bytes with "..." or some other similar scheme.  While simple
    enough to implement, it does add some complexity, and I have so far
    been unable to convince myself that it is warranted. So I have
    decided to drop these patches for now.
- Link to v1: https://lore.kernel.org/r/20240705-bnxt-str-v1-0-bafc769ed89e@kernel.org

---
Simon Horman (2):
      bnxt_en: Extend maximum length of version string by 1 byte
      bnxt_en: avoid truncation of per rx run debugfs filename

 drivers/net/ethernet/broadcom/bnxt/bnxt_debugfs.c | 4 ++--
 drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

base-commit: dd1bf9f9df156b43e5122f90d97ac3f59a1a5621