From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f178.google.com (mail-pl1-f178.google.com [209.85.214.178]) (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 A37083BE17B for ; Tue, 18 Aug 2026 11:17:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787051852; cv=none; b=gE5fSOS3gcWBmr3tpyFu+26kSJUlwv6/9nuaYy/iEVKVjjL2Rh7tPVrMZkc0i2qi3jcg1uy3AHp6wvlZuHPKs96TOk+hTUJ5A9T2MWHKNECmkSnvixYNRshrAai9JpumvduIuPjCjp4lieAphTUHGEXXH5cUYPh3F3O2XMKHuD4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787051852; c=relaxed/simple; bh=tL7b7vwmvwNYbRhTkUHjFj5QgbKWlQf9c4RVmRdSyJI=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=VCqC8a8AIy0NXaIH1GY3xNlCgUSKkNnlPyl4Aov8eLpCFUsHRcqu9bGCdJGmo6pqJjWhuzXXfMhQkn//110L1VU7MrnB9ZzSvNofWMKAIt1yBvlSyz/CGrZ31GIZudPa74/IwWnqf+o8sVEnMPPpT+TIalcGoNZwShvZ0pUt+Cs= 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=WtluEA/B; arc=none smtp.client-ip=209.85.214.178 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="WtluEA/B" Received: by mail-pl1-f178.google.com with SMTP id d9443c01a7336-2caed617615so59099075ad.3 for ; Tue, 18 Aug 2026 04:17:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787051851; x=1787656651; 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:content-type; bh=vP5hOVQLKudYfbAgDB1YvOAq7rzYSIU1ErM8W86kpjo=; b=WtluEA/BlXejyYMi4CoclplK3vYZ5O7XLkGWWJR0f4UGxKXDouFsyDqLIgQVCjZMHZ H2UJa3aqezCFK6BhacnkzJZsmWGetTV2ZUa7mc2QC36q5StkaONS1gU3HaYuFeu8czRV MSEPaR1lkcR9l+6VxQa41BCK8ueHgBKEBC2n9yMkbXY8k1HQqNzorUX/d+eLJncbHpKQ YnDeRObQO/bCBiwHOY2KMH2Wb4rU05YZZn0iXCJQurnpSLvilLqt633NLLsAhvFFVJik cJ9TLDj1efOybnp8IvXVQPPoiVuUDWNxC82VR3/zoTvE3hdl6TzEkyJUk8gcWY7ju4iU AnkQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787051851; x=1787656651; 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:content-type; bh=vP5hOVQLKudYfbAgDB1YvOAq7rzYSIU1ErM8W86kpjo=; b=DeZrIy6k7uIYBa5BL4PWZhkXbdHv27lhX5/kcos8XXjrxnA/0B/0cKJocqyD55uscK uei2z8Q1UbLu6617nnQ1Ltx7WCIvMxclAsWW8BaSBLVDO4kFd2+99mtnXpqFwTrdPrcj zq55fRmNH8OAuGX+KwfbfIWy775uZwCjqYQxJmleY+dBeW74JxgXo97z6LUOO7Z1W0Zx XOIpMg3OdBqqIs8xYpj+AHtmqUcQ9EDWPw86PJ/wm08THGMR6OT6+UNTtz6wSYPcYtGY CqNVU3qNL3W2AUJruDCbqpOi2RdDlXDgctpZUx/vSuxHv8xl/gboKYf2TTpotWLCOgJL W/Sw== X-Forwarded-Encrypted: i=1; AHgh+RpWm6W2r2m3uA6gPonu70mSAKaYF2avFJEyZsogCbudNx9RiTWy0DJcHWgJ4FfSOKMRSlKh8vUw50rR7dY=@vger.kernel.org X-Gm-Message-State: AOJu0YzTkE/Yk3iNPzGYvnQ7J3qvt1VLVo98/eKKL/yzRpasbTvXfOqy 1xp9Qgf/npzXKlY8i9uEu6p4yVLXapuZzqBLhkurt4eDshshdhx4Vr0F X-Gm-Gg: AR+sD10xbxcWh5ob2OBSgKjlAJ8jknxdqW5djujhFRcrg7DDdckS1vEb7Y0Qc8HS6wo AcI7GoDwfrNQTj10HXzsS8Ct02T6TMAGgfbwt0e/mqwVQSdV5t0Q7fX5RIIqlp35ayi/Eo4Wj+h V7fiG4QYH4KGnvl5GMCRjMgYeTiTAiSRcXHd9VwnWh2cAXfVssLIweA8t665xuSd8u/vvwvx2MU x6Jk8B6fiesew2D9Y3odmmIgFv6dGJ4Z2FE9afqM6RAbwr+m4Sqg4L57ohMe6dJvtOPzDymWjYO MnJQVoSmhVpbKyhmy3yC3JObTQiKdy07niU54chFKxbXtfWWXMrRYMNInMMjdMBmbfk2bnCiuN9 mKqm4W52VGkUnKnLosmUGIdDraGdCCm8fU0Qp+bmDOrGcSTOm+CaF+RL3Qcfe8GzMsALClYX0eV RjTxklD1XUOxpScDJYZtofzyfg2VPMf2iIf9Bj1onFpDLnbPOPf66sF1SxbzrWksyO40GsLoiCE Bc= X-Received: by 2002:a17:90b:4ac8:b0:38e:11ba:992c with SMTP id 98e67ed59e1d1-3933e5d22acmr30926781a91.12.1787051850858; Tue, 18 Aug 2026 04:17:30 -0700 (PDT) Received: from ubuntu.. ([49.205.39.116]) by smtp.googlemail.com with ESMTPSA id 5a478bee46e88-32678dd08d6sm26199017eec.4.2026.08.18.04.17.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 18 Aug 2026 04:17:29 -0700 (PDT) From: Ajith P V To: FlorianSchandinat@gmx.de, deller@gmx.de Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Ajith P V Subject: [PATCH v4] video: fbdev: via: refactor strcpy and viafb_name Date: Tue, 18 Aug 2026 11:17:22 +0000 Message-ID: <20260818111722.471340-1-ajithpv.linux@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Eliminate the single-use file-scope variable `viafb_name` and pass the "Via" string literal directly to strcpy(). Since "Via" is a literal constant, the compiler safely executes compile-time bounds checking during fortify verification. This satisfies the security initiative requirements while minimizing code complexity. Signed-off-by: Ajith P V --- v4: - Drop the variable entirely and use the literal string directly in strcpy() as suggested by Helge Deller. v3: - Restore the original "Via" string value to prevent userspace ABI regressions, while keeping the 'static const char[]' optimization as suggested by Sashiko AI. v2: - Keep variable static but change it to a const array 'static const char viafb_name[] = "viafb"' as suggested by Helge Deller to let the compiler leverage compile-time checks. drivers/video/fbdev/via/viafbdev.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/video/fbdev/via/viafbdev.c b/drivers/video/fbdev/via/viafbdev.c index 80f95dac32c8..dd9374e8ad09 100644 --- a/drivers/video/fbdev/via/viafbdev.c +++ b/drivers/video/fbdev/via/viafbdev.c @@ -16,7 +16,6 @@ #define _MASTER_FILE #include "global.h" -static char *viafb_name = "Via"; static u32 pseudo_pal[17]; /* video mode */ @@ -144,7 +143,7 @@ static void viafb_setup_fixinfo(struct fb_fix_screeninfo *fix, struct viafb_par *viaparinfo) { memset(fix, 0, sizeof(struct fb_fix_screeninfo)); - strcpy(fix->id, viafb_name); + strcpy(fix->id, "Via"); fix->smem_start = viaparinfo->fbmem; fix->smem_len = viaparinfo->fbmem_free; -- 2.43.0