From patchwork Tue Oct 31 08:58:32 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dan Carpenter X-Patchwork-Id: 740195 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5B720C4332F for ; Tue, 31 Oct 2023 08:58:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235463AbjJaI6p (ORCPT ); Tue, 31 Oct 2023 04:58:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47112 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235461AbjJaI6k (ORCPT ); Tue, 31 Oct 2023 04:58:40 -0400 Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 313AFED for ; Tue, 31 Oct 2023 01:58:38 -0700 (PDT) Received: by mail-lf1-x132.google.com with SMTP id 2adb3069b0e04-507a55302e0so7670473e87.0 for ; Tue, 31 Oct 2023 01:58:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1698742716; x=1699347516; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=sXd3kUS5wHNAxeLEX9Wod1It6j8fLEdxdydd9z/gl3k=; b=U1jiqDVXHKkahJdGdvgdhW35pLmIJxtXq5pnwqnaTg+8ec2sjZRS0sH4ApwP+df0Du 18+mS0CZvkI5bivO0IuU8BQcBbaOly0fZtcWvXulvUBkflGq/bE7HiOc+xT6F/t6urA2 Th3m7Efw1X7HyO50E2yOrM+bl7Gyom+wpHqyGnTU4wwAbcWweOzs5o8k/DClV0+EFGBG JrwXT0x8c4yNd9Ssi8PsROsY/oKvwtbenXPp1AV7LQ1hVYlTEeadTAKKPfSom65gLirJ woUTSQneRopXsG2Dgd5EYPyK/oTy56A9m+kqIMXmub9hsWUf+ToeKznFh9RVIMha6ORs 32Kg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1698742716; x=1699347516; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=sXd3kUS5wHNAxeLEX9Wod1It6j8fLEdxdydd9z/gl3k=; b=oZ8d30u91ORLwbdzkua+4rXKzf25qB+6zuFFugQLIKllsJ/Uzs27Jp++1/I9Dzr4Fh GngVHSaaaaBKh6IqIHJY8wwfkfOoAvVjKWKj0fDG51Kjh+Ro5isC/HYhcxbt68Z6nFyF wDCE3hs2j9pwVVfIazkzVEmnff/CsTen67OVGqVMq2bYMMzTr6yErlSVorcwC3eyLMri kJckcuBOLx0MOEDTymxyBXa+sp7PR+/MvkXkfI28oIfBdNvpkIyzl6f4a74XUQVIH413 nBX6xctd7mLO7hn8Ylu8XWsS1V8LrntaN4I1F9povK+GgH+f6Gws85j0pw6kYyTKH/vD aMOg== X-Gm-Message-State: AOJu0Yx20kJj8L04YudtiHMPqeR7zp853wS7bztF1CZn4IhWIh2H/LO5 Th+jxTVKVrXJkfFYi/JE2XY5Ng== X-Google-Smtp-Source: AGHT+IEFqBdMG3El7Gl4hxNtoAhylcDlcZMKG3r/NcM6r+dI2+dtHsm53rfSB/1p1rzQYinkmTsSbg== X-Received: by 2002:ac2:522c:0:b0:507:9dfd:f846 with SMTP id i12-20020ac2522c000000b005079dfdf846mr8472138lfl.69.1698742716351; Tue, 31 Oct 2023 01:58:36 -0700 (PDT) Received: from localhost ([102.36.222.112]) by smtp.gmail.com with ESMTPSA id c8-20020adfef48000000b0032f7c563ffasm983675wrp.36.2023.10.31.01.58.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 31 Oct 2023 01:58:36 -0700 (PDT) Date: Tue, 31 Oct 2023 11:58:32 +0300 From: Dan Carpenter To: Ciunas Bennett Cc: Giovanni Cabiddu , Herbert Xu , "David S. Miller" , Adam Guerin , Andy Shevchenko , Damian Muszynski , Tom Zanussi , Shashank Gupta , Tero Kristo , qat-linux@intel.com, linux-crypto@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [PATCH] crypto: qat - prevent underflow in rp2srv_store() Message-ID: <3fb31247-5f9c-4dba-a8b7-5d653c6509b6@moroto.mountain> MIME-Version: 1.0 Content-Disposition: inline X-Mailer: git-send-email haha only kidding Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org The "ring" variable has an upper bounds check but nothing checks for negatives. This code uses kstrtouint() already and it was obviously intended to be declared as unsigned int. Make it so. Fixes: dbc8876dd873 ("crypto: qat - add rp2svc sysfs attribute") Signed-off-by: Dan Carpenter Acked-by: Giovanni Cabiddu --- drivers/crypto/intel/qat/qat_common/adf_sysfs.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/intel/qat/qat_common/adf_sysfs.c b/drivers/crypto/intel/qat/qat_common/adf_sysfs.c index ddffc98119c6..6f0b3629da13 100644 --- a/drivers/crypto/intel/qat/qat_common/adf_sysfs.c +++ b/drivers/crypto/intel/qat/qat_common/adf_sysfs.c @@ -242,7 +242,8 @@ static ssize_t rp2srv_store(struct device *dev, struct device_attribute *attr, const char *buf, size_t count) { struct adf_accel_dev *accel_dev; - int ring, num_rings, ret; + int num_rings, ret; + unsigned int ring; accel_dev = adf_devmgr_pci_to_accel_dev(to_pci_dev(dev)); if (!accel_dev)