From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oa2-f12.google.com (mail-oa2-f12.google.com [74.125.231.76]) (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 3EEB34CC62E for ; Thu, 24 Sep 2026 20:35:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.231.76 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790282124; cv=none; b=g+JsHEdZMAETNtABnDnihIXYUqy+wMbio+GxGPTa/a9LNE8dZyXUU4Br0fJQ3hK2eY5uoJw01SA0Be0KCzaPKWHxpbj0GB01m8VpHw2X0c9Ml7PIUTeJEp3OBZfNAlIhuSMYxUYk15sfgoRkvKev9AmP3BZhtAuS+YBEGZxCkZY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790282124; c=relaxed/simple; bh=Ezdlx+UeRBSR/f6nrz4LesZPCELyFYKZoA7wcmaGEzc=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=YR+Wl6pdG7mkle1yhEi4HX7UR0rM4IvU/GDBtAue9ssbEKkIozEUGAe5GcPaqqMt0Yz2uRfj0WpELYIc8SXq52Ejhyrd5hcuMBOcxGk54S64BpFHOqUDgq8QT0iU+NNF/JFFvKAyUi8vrKbHP1QUcnz9vPfIP9XdbFOYeGHvGnk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cloudflare.com; spf=pass smtp.mailfrom=cloudflare.com; dkim=pass (2048-bit key) header.d=cloudflare.com header.i=@cloudflare.com header.b=OQLzgFZ7; arc=none smtp.client-ip=74.125.231.76 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cloudflare.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cloudflare.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=cloudflare.com header.i=@cloudflare.com header.b="OQLzgFZ7" Received: by mail-oa2-f12.google.com with SMTP id 586e51a60fabf-466cc88a0d7so238722fac.2 for ; Thu, 24 Sep 2026 13:35:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloudflare.com; s=google09082023; t=1790282117; x=1790886917; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=+deUfdU77FGkM4KhZU62y6wx4HZXqi9klM76GTBpRK0=; b=OQLzgFZ76R8SdvMgiG03fk4vLkTXvd6GRNqUUm3aoJbsQyfWXpteXNooZYwZ0VJPBL x9plmvkYPM99+p9iwRhLCF5Aba4Fj18/o3G5sRK5Sv6Pzgh0M5wqndzYVaJ0aoq7omFK dD+V3+i5K2KpY5FU//81KeGwS3aNRDmoe/wIjFNR8QmcEqMBfPTlwsdYaeqvmHbHqmA5 /3npuEljJbhmOAj2aWfG5/4nhmEZHFyFOgztJOpvyJ4BFd3zmsdaZiYt88wOlVY/1fdr lYiiaTbPmxtlxiAp2rmC2mfPr6t/C0sgfpNqSMiBYPFAy3exfe5zryoEguPuHAHOIaFj CBKg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790282117; x=1790886917; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=+deUfdU77FGkM4KhZU62y6wx4HZXqi9klM76GTBpRK0=; b=kaIgLv/GxOqzZ8ECsDxr6AHiyEc/GDDiiXuvbSYGkxITMJR8qjlqH6KZTrAkTCgfe6 Aye09jSPwoFe1N/hyD7fmw5dsFy+5Qi3TY9BWNv+e8YlC4koN4FbWb/Hf/YLhcrDXxKp gH9box8O/thZ+Dom1f9xCnM1DoaG7GBJJMK7i99udyyphqCK7hUlTmAhYfiH0uR81wpr P/V23dCRqbYGIbTw4Mb2KP13h/3QPnayeEXVHUSweh0Wib85DHe1VYNF1fw4wyJ6gmvZ DghEiyC4LddnTXypX29MzmBQOaYTySbEidXRezWWePnCNUYUQ+aC8gOfY1TEl9h6naln ffhg== X-Forwarded-Encrypted: i=1; AKwUvByUykaeDmEyLCBiNBYrrVbTmrJ2mVMZbtkCb1LZFrsOxicmJmDwlSIFZi5Gi9ktfZUJEeay/2aMfEFMrck=@vger.kernel.org X-Gm-Message-State: AFuF++nhuMz4DdlQvVcL8PMC2exwhylMcplo2zmI1Lmsp3RRsXdh36EP d9IHV1ViDg8t9lsrLhHVkmg+pKctc9NpfjYZhLSs5ng5+nOXDdImbbV+PzftHZSBnpc= X-Gm-Gg: AYBFou2aWCDmyQTTswyBzcjI/mYhCaDB13H0vPl91WMkPindoq+TKTgHWII/oI7s+7h DoRNQFZ3XQQRUWClNkh6KUqZEGqMK4S+k6bBZ2iCWrq6TK2bYw0wkCyuE+8iJ+jvGZlCznVwUiB bBWnhE7Vuh2jdhuAHHSMXlHViul3n6v4xR8jKU5EXfTz/+ZIqrA0uYa9HxMDAvoQQNPC2exjDGW ipFWEVm/1NPhlS2P3aV8axofZzeL8emt54m5rxavxAD7OTtRNg3wfkYtN+FsVcQy8ov9QCzRjTV z8RFpam1xedgBc+pOP8aGKteqxylnq52eEgeaw4F/pyVG3XY/GCqZCyBcJjn4HdjWOtut7Dp9Lw +8CGFwZ/DAEuhJixDIyLC5XW4OlNnA05t8cQ+APJjF5KaED01WxsLQN51ReTeRvne0Y6lxxSDUn z7P0MNqUWkm8L/tmIfMZN7VJriqJhycbicxAOBuJAj3vT8fYnWSXWJ/904 X-Received: by 2002:a05:6870:3263:b0:48f:e107:317b with SMTP id 586e51a60fabf-491e6efd09bmr4096695fac.46.1790282116700; Thu, 24 Sep 2026 13:35:16 -0700 (PDT) Received: from [127.0.1.1] ([2a09:bac6:947f:1cd2::2df:f]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-49335011458sm254515fac.4.2026.09.24.13.35.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 24 Sep 2026 13:35:15 -0700 (PDT) From: Frederick Lawler Date: Thu, 24 Sep 2026 15:34:59 -0500 Subject: [PATCH v4 2/2] integrity: Replace integrity_audit_message() with integrity_audit_msg() Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20260924-report-hash-error-v4-2-196ca6350619@cloudflare.com> References: <20260924-report-hash-error-v4-0-196ca6350619@cloudflare.com> In-Reply-To: <20260924-report-hash-error-v4-0-196ca6350619@cloudflare.com> To: Mimi Zohar , Roberto Sassu , Dmitry Kasatkin , Eric Snowberg , Paul Moore , James Morris , "Serge E. Hallyn" Cc: linux-integrity@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-team@cloudflare.com, Enrico Bravi , Frederick Lawler X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=4050; i=fred@cloudflare.com; h=from:subject:message-id; bh=Ezdlx+UeRBSR/f6nrz4LesZPCELyFYKZoA7wcmaGEzc=; b=owEBbQKS/ZANAwAKAasltHYDktNtAcsmYgBqtYl7XvGk1MJfTdKnrYbu8pV2c9CadsSpdb8fC uBHarKmHcWJAjMEAAEKAB0WIQTLNBqMVmu1PHvjOe2rJbR2A5LTbQUCarWJewAKCRCrJbR2A5LT bXxXEACUA/aczjDQzxajR+a0kcLYZnuvxzY6UzOsfjftWnjRH4QhB3hH9PJRztqXTvt0a1cDSB8 YhL0+4jayAQdqXLhla0SKqAmOM50iS1GW0UOxfegOtSg89vOVCpSOyUah1nTxbN4dNNQmn14hOu 8Mo0Pd2lRRrjPJqxX0Lk4GlSpMt4BiN1VgXgJ/d0Lafh/VomRpFjBMWop5Dy8oNhtdO2vzny31r idzKhC+7IQSc0Y7wgQTUSM+aIzuS4vg4khe8aXr3RsLoIH2drakS7I/R3OzyPldD8E8dm9LY4FF FE/7FU2OqgdW5MYUHJvZYa7qT2ZeigppOKvqTkBQpvlPGR/5YaCKptxTi+4YEUwLeQ6mcycBf1W tYuojIFFU9zbrXrFNlitGgNflLq1z4yfuUwI87YgvIln7zai1zjiU/ujHeUkCdWvNehst91+XkQ PedRrV+fou0KzRV22626vJtQUREP6ByTytyzNItgC3p/EmRe6wJ24L4wBokjgaL41rB4GjOma64 iYTMyjMXkcTPD9TC7T4z48x00bzz7koEX+BCWeJLdVMOVgUAcX121Z+0xlYIhw31zFk6mQbdsVn nVBpv2zUEX+5td3TV/h1gJ31JVJEQN+xWH0uX6kJyayislvRBGHX3vgjgIn1bfelB9DQqNS3+NF EwcJlLz8nN2phUw== X-Developer-Key: i=fred@cloudflare.com; a=openpgp; fpr=CB341A8C566BB53C7BE339EDAB25B4760392D36D Now that integrity_audit_msg() accepts an errno parameter, it is equivalent to integrity_audit_message(). Replace the remaining integrity_audit_message() call sites with integrity_audit_msg() and remove integrity_audit_message(). Signed-off-by: Frederick Lawler --- security/integrity/ima/ima_main.c | 6 +++--- security/integrity/ima/ima_queue_keys.c | 8 ++++---- security/integrity/integrity.h | 13 ------------- security/integrity/integrity_audit.c | 9 --------- 4 files changed, 7 insertions(+), 29 deletions(-) diff --git a/security/integrity/ima/ima_main.c b/security/integrity/ima/ima_main.c index 9e80f40680d7be08d14d26ba48ed334d4e4ab10c..bd9997b01633d6235fc2fcaa9bef1931d6ba7247 100644 --- a/security/integrity/ima/ima_main.c +++ b/security/integrity/ima/ima_main.c @@ -1196,9 +1196,9 @@ int process_buffer_measurement(struct mnt_idmap *idmap, out: if (ret < 0) - integrity_audit_message(AUDIT_INTEGRITY_PCR, NULL, eventname, - func_measure_str(func), - audit_cause, ret, 0, ret); + integrity_audit_msg(AUDIT_INTEGRITY_PCR, NULL, eventname, + func_measure_str(func), + audit_cause, ret, 0, ret); return ret; } diff --git a/security/integrity/ima/ima_queue_keys.c b/security/integrity/ima/ima_queue_keys.c index b5ed33cbb272fdfaf7f5e945137c56e46e0d7db1..8adc8863ad5a540bf0c047ba47cc4817fcb46e9a 100644 --- a/security/integrity/ima/ima_queue_keys.c +++ b/security/integrity/ima/ima_queue_keys.c @@ -90,10 +90,10 @@ static struct ima_key_entry *ima_alloc_key_entry(struct key *keyring, out: if (rc) { - integrity_audit_message(AUDIT_INTEGRITY_PCR, NULL, - keyring->description, - func_measure_str(KEY_CHECK), - audit_cause, rc, 0, rc); + integrity_audit_msg(AUDIT_INTEGRITY_PCR, NULL, + keyring->description, + func_measure_str(KEY_CHECK), + audit_cause, rc, 0, rc); ima_free_key_entry(entry); entry = NULL; } diff --git a/security/integrity/integrity.h b/security/integrity/integrity.h index 2ab6b7e34e7ee2dd29ae03ae1fb45fa26d9c032a..52c5ec881cca468bde72c885cffa1c564877735e 100644 --- a/security/integrity/integrity.h +++ b/security/integrity/integrity.h @@ -221,11 +221,6 @@ void integrity_audit_msg(int audit_msgno, struct inode *inode, const char *cause, int result, int info, int errno); -void integrity_audit_message(int audit_msgno, struct inode *inode, - const unsigned char *fname, const char *op, - const char *cause, int result, int info, - int errno); - static inline struct audit_buffer * integrity_audit_log_start(struct audit_context *ctx, gfp_t gfp_mask, int type) { @@ -240,14 +235,6 @@ static inline void integrity_audit_msg(int audit_msgno, struct inode *inode, { } -static inline void integrity_audit_message(int audit_msgno, - struct inode *inode, - const unsigned char *fname, - const char *op, const char *cause, - int result, int info, int errno) -{ -} - static inline struct audit_buffer * integrity_audit_log_start(struct audit_context *ctx, gfp_t gfp_mask, int type) { diff --git a/security/integrity/integrity_audit.c b/security/integrity/integrity_audit.c index 2dd2d7e3916e7bf35c753450315ff22e2d0292b7..22106aaeb0df21fef0990b80611dcd13cba112be 100644 --- a/security/integrity/integrity_audit.c +++ b/security/integrity/integrity_audit.c @@ -29,15 +29,6 @@ void integrity_audit_msg(int audit_msgno, struct inode *inode, const unsigned char *fname, const char *op, const char *cause, int result, int audit_info, int errno) -{ - integrity_audit_message(audit_msgno, inode, fname, op, cause, - result, audit_info, errno); -} - -void integrity_audit_message(int audit_msgno, struct inode *inode, - const unsigned char *fname, const char *op, - const char *cause, int result, int audit_info, - int errno) { struct audit_buffer *ab; char name[TASK_COMM_LEN]; -- 2.43.0