From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3C0F37260F for ; Sun, 5 Apr 2026 22:20:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775427619; cv=none; b=TMQhSZALRB50mqee10xkAER6YhuQvsElzfWONeU47yD7XQuC6YAqbaqCYNYzMhrQyxljyLQPsrPe40zpy11UJe3VFwb2kp3QhYbhA97wJ+A+us40HCEM6BMNwKVuvg4FCCSNAlGqnQo6GngKwcxeSnNO3jOpbLxoPNGMyL0BLxk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775427619; c=relaxed/simple; bh=6r99neyde3b0isfHa5GQwCkPtLceRO7kB6KTmuQSUL4=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=W5+NV1qDbz05oM/PpjjAZBVtkc5fmUF1BY1sB7KVTC9X5CymHP/EgfTWJs0YPVT0LnvvN260lhxLUDARW4WXDu0cIaVGMVD27/hAKbyA0hBTwJQ68R9kzsxUIi+88sauUnL7fBKY63KGLqZJBUx/TsWVLbfGcDcHenlCF3G29+I= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=GvE6F74a; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="GvE6F74a" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-488af96f6b2so6714275e9.0 for ; Sun, 05 Apr 2026 15:20:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775427617; x=1776032417; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=UYtvotg7WC0VoukV+5DSJczX5Hfpm/Wf8rZ92IGGmqM=; b=GvE6F74aXbCJmrkEbq0ZwuAj8jigxOq4W305hjGyqAuwzgTVoOOK0G28jRrqzVwbRS 20UnR0jTgPwOv0rajBVFT1W13MHmsukSk4NnFFuLr6bpuRCPDu/jtOZTfG1yRcNFCESr R08VGZ9LuegZlZqEsjgjtDBzMXOVye8cn0q1z9VTgSVVXhCydpxixdxGiQl4PsxEEnvH oNB4k67fyjUId7H/Jf2bJxVx6ADWmNkYYpcEPmCIFzEtRfKSorv4bgPoPqscq2KQJOdd h5hbkDjpLyYkhhiMARsECfBD8gqBGNFSYpVKI6SHumFpOhiUSgaa6+UZgh+kHbqJ0lXE 1a4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775427617; x=1776032417; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=UYtvotg7WC0VoukV+5DSJczX5Hfpm/Wf8rZ92IGGmqM=; b=olMH8Zm7ojgecFub0hyt/Dfo/Jm2z/oVxUAJ/yJJiPgbuMqrq4f6jD2ReVjlxqi0WM yFoUEAcNZ54PwMqKWRGaa2s6gUBKlXRME/ofXBRNjTVokYeC3ocuT0TxuXixnEz2NzF7 l3VAoYTJ2TSsyD4uLQuM68OUp9fCC9QGbAFznQ4tVroehx0ZCKh9mUQ2knXGnqQx5ZXE dCZUMQ4hvo34nv655vNCJUZqzpiAHPpsZd9Xn1zQIfe21qpa1OOnzrPsXYPz+iIL7gQG XJh1ivy2PkQvFNMiMwJXknZbTztk9002yGart1LMFuz2oiDEDjkGAg8z+UeIw3khdRIY bsKw== X-Forwarded-Encrypted: i=1; AJvYcCUpMrdgS+bHUQuGQIUvz4CGURPt3n5IAfQC6I/xFEPqfE9Qh04cNpY2PFkFVD/F/OOtcNmX+nQ3wxGgf1U=@vger.kernel.org X-Gm-Message-State: AOJu0YxvFmCs6qqW65fIfKqJ/LW8DSqhSmMfaFKdgUMCcwM9lqLzm/R2 f4x6TYmxyEiy61d7qXfe9BTVZ9+rsbB7XXuGAXbpZS1XiLPjJygEx2Iw X-Gm-Gg: AeBDiev2vyXihstru9ahoAYZLRk8svKcQrmCLM23RtqG5TS+PhcMOAQpYqhzYHt9Q7L VekeFRtpQptgnWJwyaOdQWjOJVpYsdFxwSy9yZ5AUwVmVIrQ8f/+rF5rtz66OYEfORmBrQ265vf yU8CFWkX2btOG4PLZB2j61FO1HmE60yHk307FVM6U3OmSUY7Do8b/84/TUUJwHF5Ev7dcZT92Yq lOMMqCg5ESPkpRw8Tsj7XLBQ5Bz9TLRVSxnwLJmvKo65EVXDY5MccRNjJbTHmZAbl0Cw4S0pb4I VPZWOTOARxa3laJFsRzFIcFZ/RELooPD51Pi2N1QMya5UR9AANuOsBb701qOscqEJ5rj98P6Azv W2lk0jgHFyXuWsOcaJBTxuRKB/lAl6nDI2dKFHWgB6qEoQytgGGURScXaykU67kH9GrNCJngNh3 u48WNar1+R4qxyOSJVNyBYuTgL X-Received: by 2002:a05:600c:4ec7:b0:485:35a4:939f with SMTP id 5b1f17b1804b1-488997ebfbamr136868985e9.28.1775427616390; Sun, 05 Apr 2026 15:20:16 -0700 (PDT) Received: from rpi5.lan ([37.228.206.31]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4887e829c43sm413182325e9.5.2026.04.05.15.20.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 05 Apr 2026 15:20:15 -0700 (PDT) From: Fabio Baltieri To: Jiawen Wu , Mengyuan Lou , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman Cc: Fabio Baltieri , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] net: txgbe: leave space for null terminators on property_entry Date: Sun, 5 Apr 2026 23:20:13 +0100 Message-ID: <20260405222013.5347-1-fabio.baltieri@gmail.com> X-Mailer: git-send-email 2.47.3 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Lists of struct property_entry are supposed to be terminated with an empty property, this driver currently seems to be allocating exactly the amount of entry used. Change the struct definition to leave an extra element for all property_entry. Signed-off-by: Fabio Baltieri --- Hi, bumped into this while studying the code, looks like the struct has been allocated without space for terminators, guess the top ones just end up using the bottom props as well but I'm surprised this does not crash at some point. Build test only, don't have any hardware for this, let me know if I'm missing something here. Cheers, Fabio drivers/net/ethernet/wangxun/txgbe/txgbe_type.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/wangxun/txgbe/txgbe_type.h b/drivers/net/ethernet/wangxun/txgbe/txgbe_type.h index 82433e9cb0e3..6b05f32b4a01 100644 --- a/drivers/net/ethernet/wangxun/txgbe/txgbe_type.h +++ b/drivers/net/ethernet/wangxun/txgbe/txgbe_type.h @@ -424,10 +424,10 @@ struct txgbe_nodes { char i2c_name[32]; char sfp_name[32]; char phylink_name[32]; - struct property_entry gpio_props[1]; - struct property_entry i2c_props[3]; - struct property_entry sfp_props[8]; - struct property_entry phylink_props[2]; + struct property_entry gpio_props[2]; + struct property_entry i2c_props[4]; + struct property_entry sfp_props[9]; + struct property_entry phylink_props[3]; struct software_node_ref_args i2c_ref[1]; struct software_node_ref_args gpio0_ref[1]; struct software_node_ref_args gpio1_ref[1]; -- 2.47.3