From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7F0B51DC06F; Tue, 3 Sep 2024 20:45:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725396333; cv=none; b=knzNrROFVDkg71FY6fzfnI/D168JbQXIw+CG5Q7do7j/SGHAc3YE7jiRXR2zWgDUoEZ4rgOim/XxyoIscNP+FzPfiop3jY6HDBqiF9sSHSjrW+j67x0uYEQrhpOdndskmtzmP9erXkYr5WvkQnLj6ez7Zthk6qb+oVojFXh/aZ4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725396333; c=relaxed/simple; bh=yF0rmGrroWRUqwwwZYSX9vlCZ7YX8ljC+aLaMwGf8ws=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=pijdLo9jKXx4nk8/sDwSk/xY/geOXXZSBFgbE5t0acUy5g4hUXwMXaIiZW3AYRObtvf9n5N/DeCGnvcyZg9jckpV234a6/OaMu/7faG83/USYjdQrZKyJlP6fymvuzNnU/+52fZB2SUdfVe8RSsNfqNhPgRCFTasZeDT7QZfTfs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=CL9GgR9v; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="CL9GgR9v" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 31890C4CEC5; Tue, 3 Sep 2024 20:45:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1725396333; bh=yF0rmGrroWRUqwwwZYSX9vlCZ7YX8ljC+aLaMwGf8ws=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=CL9GgR9vwxJP62bF/tsXnSoVcddDNWR4Dzrr8gmppt8KcknGmv+To/UN3K3sRjdvS wGeRwg2nMiBsnHjuGmQ0E6Rs8Nz8Q6g2D9SsFT9RAnZxP7uVlcBDE1xXAekbW4WhJo EmoWl28iW7iGQ+dcIfAcunpFhKEdTpksuurVJkJgLyItfo4p7rY8MifEnrhenbPXpH HEH/iGc8VKdtuK3eB1YD2js9/Jxqp3zfaGNyOWBAKm0yXrEpZtXGKlEC/aU5U7pmlr /ukoheNQjHRuKbHWt+XTNnd7rNt8U6kUYoJJlKuGXAlVxGz/Nkg/MHGnUCYFTiwLyo ty3Y2k3ag3DEg== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Sherry Yang , Justin Tee , "Martin K . Petersen" , Sasha Levin , james.smart@broadcom.com, dick.kennedy@broadcom.com, James.Bottomley@HansenPartnership.com, linux-scsi@vger.kernel.org Subject: [PATCH AUTOSEL 6.1 06/17] scsi: lpfc: Fix overflow build issue Date: Tue, 3 Sep 2024 15:25:20 -0400 Message-ID: <20240903192600.1108046-6-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240903192600.1108046-1-sashal@kernel.org> References: <20240903192600.1108046-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.1.107 Content-Transfer-Encoding: 8bit From: Sherry Yang [ Upstream commit 3417c9574e368f0330637505f00d3814ca8854d2 ] Build failed while enabling "CONFIG_GCOV_KERNEL=y" and "CONFIG_GCOV_PROFILE_ALL=y" with following error: BUILDSTDERR: drivers/scsi/lpfc/lpfc_bsg.c: In function 'lpfc_get_cgnbuf_info': BUILDSTDERR: ./include/linux/fortify-string.h:114:33: error: '__builtin_memcpy' accessing 18446744073709551615 bytes at offsets 0 and 0 overlaps 9223372036854775807 bytes at offset -9223372036854775808 [-Werror=restrict] BUILDSTDERR: 114 | #define __underlying_memcpy __builtin_memcpy BUILDSTDERR: | ^ BUILDSTDERR: ./include/linux/fortify-string.h:637:9: note: in expansion of macro '__underlying_memcpy' BUILDSTDERR: 637 | __underlying_##op(p, q, __fortify_size); \ BUILDSTDERR: | ^~~~~~~~~~~~~ BUILDSTDERR: ./include/linux/fortify-string.h:682:26: note: in expansion of macro '__fortify_memcpy_chk' BUILDSTDERR: 682 | #define memcpy(p, q, s) __fortify_memcpy_chk(p, q, s, \ BUILDSTDERR: | ^~~~~~~~~~~~~~~~~~~~ BUILDSTDERR: drivers/scsi/lpfc/lpfc_bsg.c:5468:9: note: in expansion of macro 'memcpy' BUILDSTDERR: 5468 | memcpy(cgn_buff, cp, cinfosz); BUILDSTDERR: | ^~~~~~ This happens from the commit 06bb7fc0feee ("kbuild: turn on -Wrestrict by default"). Address this issue by using size_t type. Signed-off-by: Sherry Yang Link: https://lore.kernel.org/r/20240821065131.1180791-1-sherry.yang@oracle.com Reviewed-by: Justin Tee Signed-off-by: Martin K. Petersen Signed-off-by: Sasha Levin --- drivers/scsi/lpfc/lpfc_bsg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/lpfc/lpfc_bsg.c b/drivers/scsi/lpfc/lpfc_bsg.c index 2373dad016033..fc300febe9140 100644 --- a/drivers/scsi/lpfc/lpfc_bsg.c +++ b/drivers/scsi/lpfc/lpfc_bsg.c @@ -5409,7 +5409,7 @@ lpfc_get_cgnbuf_info(struct bsg_job *job) struct get_cgnbuf_info_req *cgnbuf_req; struct lpfc_cgn_info *cp; uint8_t *cgn_buff; - int size, cinfosz; + size_t size, cinfosz; int rc = 0; if (job->request_len < sizeof(struct fc_bsg_request) + -- 2.43.0