From nobody Sat Feb 7 16:00:23 2026 Received: from mail-pg1-f202.google.com (mail-pg1-f202.google.com [209.85.215.202]) (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 10C1813E02A for ; Thu, 8 Jan 2026 00:26:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767831991; cv=none; b=p3snZtNYrCbEff80L57lHiGBWgSG/d+Afg3CnXjq/tfBEBfse7zLk9K5VR4ZjasudhqDaTNnYS/A5BWhrz3TN5EABtMms+qrEh8tEob7NsUkFLnxjLxrmafviFvEBhvo+MBIOn/y/rNzOyeVWXUS6ZoWkUe/+RtP0W9b/vj0hiA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767831991; c=relaxed/simple; bh=k3sAb0XeYbzvwvAPQHAnYzp7msyQ9i69ex5WJgAAaXE=; h=Date:Mime-Version:Message-ID:Subject:From:To:Cc:Content-Type; b=FdYELK0F4jO6HR+4OyVr36USb9/yTtz0ai8p8RTNPu/Bxas+k3J3EWFNjrrW6BRX/pDyMEKEfG+lpzmAWPF/UdtrxbB9Bdg+iMPJjUV9RbNNPuJsalPHsu6Ku26GnLHnYXekyRPw8uMxe+q1i+J8TcaPETDid4zOgSfLBWTca0E= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=jUeTzKgE; arc=none smtp.client-ip=209.85.215.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--irogers.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="jUeTzKgE" Received: by mail-pg1-f202.google.com with SMTP id 41be03b00d2f7-b630b4d8d52so1899938a12.3 for ; Wed, 07 Jan 2026 16:26:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1767831989; x=1768436789; darn=vger.kernel.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=Rr1Dru9stk8d467rZzPtV2sYPWJ4jerVHwr5PFdH2m0=; b=jUeTzKgEOlMr9t6ZdSb+0wB4+GmUMz5CovNfBh4xH6PpEUApx+lEHNR21BAQe4q0th yQ3y7/8lipU9OQ8madsPkVGV5TbNnvMjYRRikrcZmXvmazk3ZOW/CKvMTmjujJfSeiid cUe2c+lwTYqfNhbm5zBt/CMTBH/MIEFL2JbTnLQRZPIVUMdbcqQ7cmYuAd7ruETnkZh/ 0G2Ubfgp8tUpbQvRfgRMS+YYLHs7nSY8QpFYbFmgRKyPUwmnXbyLf+60jngXOla3cDRK wHTNzlpaxv0SL63dGfZ4SFJjAf0Yk45gBAvIr62Uzalqg41NpZULYdjglo0PiDPgXssa sptg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1767831989; x=1768436789; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=Rr1Dru9stk8d467rZzPtV2sYPWJ4jerVHwr5PFdH2m0=; b=V/ezqL+MsqtYGJepwNIPAuzOSFEvArBnUhJ8RwpNTZUbpl+Dj53w1wQkJqYR84wmIb 9pfe+FCxpBbFd+CzTrH0Aa+Xsmi5W5/tzZUCli9j1rCjiPfhRR34hNfPr7mUGa0ZL5Nz TGB86vpRISYVG2VNanWg6i09QJIjcFrBBclW6HEVQ3N6DOK0zsZfcfgUvKEEufHjbhx/ 6jIoRqJeWnH2DHxf1IUnplNx780apIU4WRi1WuZKtyDD85oU/wBm2IX1VWOvbFlM67A3 Y0+kC+NhuXWfKyscNiqZoldIglbPf2CnZV1+YLs87FeEE0GLtlSsRmwCpJdscmWBIN4J +/qA== X-Forwarded-Encrypted: i=1; AJvYcCVBaUt6Xb5EcmYQ6sONw2foO+/SpJkS/YkTLbwcJvs3loHjivHDG1ldkxUJbAQsHCp2oRvjXMS37H8SE6U=@vger.kernel.org X-Gm-Message-State: AOJu0YzFYoMqt/ldOox+qqjUzfkrc2JnK1t91sCWpXT937RaE+HUqffY bPdoHC/W9cSSxektFPSdi8kNLo/23SLACWhB44FULqzuWCen2MzUcoMZ6Hd5b5I7R6PfibTK6lF nW7CsKS7DRA== X-Google-Smtp-Source: AGHT+IEsabmtWapDF2dFBzjM6LPsJo7u+kWkYVbY0WmxRJLm/Sg1gYng1Cja4Vn12hdS+0Hrn5kjl0CZU1Mb X-Received: from pghi13.prod.google.com ([2002:a63:e90d:0:b0:bc4:8a19:36d4]) (user=irogers job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a21:3286:b0:37e:4319:d7c8 with SMTP id adf61e73a8af0-3898f9abb78mr4207216637.75.1767831989425; Wed, 07 Jan 2026 16:26:29 -0800 (PST) Date: Wed, 7 Jan 2026 16:26:25 -0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 X-Mailer: git-send-email 2.52.0.351.gbe84eed79e-goog Message-ID: <20260108002625.333331-1-irogers@google.com> Subject: [PATCH v2] tracing: Avoid possible signed 64-bit truncation From: Ian Rogers To: Steven Rostedt , Masami Hiramatsu , Mathieu Desnoyers , linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org Cc: Ian Rogers Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" 64-bit truncation to 32-bit can result in the sign of the truncated value changing. The cmp_mod_entry is used in bsearch and so the truncation could result in an invalid search order. This would only happen were the addresses more than 2GB apart and so unlikely, but let's fix the potentially broken compare anyway. Signed-off-by: Ian Rogers Acked-by: Masami Hiramatsu (Google) --- Reduce the number of checks as suggested-by Steven Rostedt . v1: https://lore.kernel.org/lkml/20251209224024.2322124-1-irogers@google.co= m/ --- kernel/trace/trace.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 6f2148df14d9..5e2d597b4377 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -6109,10 +6109,10 @@ static int cmp_mod_entry(const void *key, const voi= d *pivot) unsigned long addr =3D (unsigned long)key; const struct trace_mod_entry *ent =3D pivot; =20 - if (addr >=3D ent[0].mod_addr && addr < ent[1].mod_addr) - return 0; - else - return addr - ent->mod_addr; + if (addr < ent[0].mod_addr) + return -1; + + return addr >=3D ent[1].mod_addr; } =20 /** --=20 2.52.0.351.gbe84eed79e-goog