From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AG47ELtiJw15uSowZmlb8P1Lbzpm+CNa6o1F49ABUu+W8JX07+BcGMz735xSaoNV6l50/L4FcR2g ARC-Seal: i=1; a=rsa-sha256; t=1521483369; cv=none; d=google.com; s=arc-20160816; b=DOxKSvnotBb9QUBh7CJiN3IN5RxdahaX5n8AfcjFRFLscfiA5Mgnurrb5D3LdPJMSQ oKHzoo8jUMv7aa5ZpPk4tq4m2KQnxrGUYYw39DArMAiZit04dPRQniwL6dm/2AWkL994 sbkjff/QrBLL73OnnHFTVl+mGVgd802Lng6H3J1PW1fk1kaHhnArZdJWg54wj91+/eUm riZYVbiA/jm7awQC/855nNHoR6cr1MvP156N1LXRzpXXXCvwCsAxC7ur4z+YTYRpRMXK fgWauI0g5Adlu02XsocSleqBkKDI0jnvxKhR/aQrUPJG7drH3hABJLY+77pdxPQCV6Sp jaiw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:user-agent:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=+0prhnoKR1LBgL/9pBJmYbcPRE5MDybbdFhLO/oX38o=; b=fHqYNUBBySWAqnL/+k/0qPGjCbaA68yfMPZ8BZwgLQ5CoFUjFIIAgUaRezH+KkjA/q X+tfHOInB+nRlhTOX8SxJfxw/BEz0hz7fL7PN9KHDzLCssIYl4hUTnUaS5jKUVMt3hgq pWvWyAlFuTW6lCf9wrLlBqHIuGCnG2LIIbydWEjsAc3I9mtA/Yg7Flm6lB+btfkL7XeU nDOkZI06fzwN7SJ7Nux6ZH4SnWBzpnEN8vQWxi063bxOsQGD9WvZWnOp9YHLF0bB3ykC FGSsGLEVh7ZRIe/+kPyHWd0+WUa3erANHr9svxE0I1onHwqSBb6g7q6Fh8rh7PhbNqXx LILQ== ARC-Authentication-Results: i=1; mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.61.202 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org Authentication-Results: mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.61.202 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Bart Van Assche , Christoph Hellwig , Hannes Reinecke , Johannes Thumshirn , "Martin K. Petersen" , Sasha Levin Subject: [PATCH 4.4 097/134] scsi: core: scsi_get_device_flags_keyed(): Always return device flags Date: Mon, 19 Mar 2018 19:06:20 +0100 Message-Id: <20180319171903.260705907@linuxfoundation.org> X-Mailer: git-send-email 2.16.2 In-Reply-To: <20180319171849.024066323@linuxfoundation.org> References: <20180319171849.024066323@linuxfoundation.org> User-Agent: quilt/0.65 X-stable: review MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-LABELS: =?utf-8?b?IlxcU2VudCI=?= X-GMAIL-THRID: =?utf-8?q?1595390945327424140?= X-GMAIL-MSGID: =?utf-8?q?1595390945327424140?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: 4.4-stable review patch. If anyone has any objections, please let me know. ------------------ From: Bart Van Assche [ Upstream commit a44c9d36509c83cf64f33b93f6ab2e63822c01eb ] Since scsi_get_device_flags_keyed() callers do not check whether or not the returned value is an error code, change that function such that it returns a flags value even if the 'key' argument is invalid. Note: since commit 28a0bc4120d3 ("scsi: sd: Implement blacklist option for WRITE SAME w/ UNMAP") bit 31 is a valid device information flag so checking whether bit 31 is set in the return value is not sufficient to tell the difference between an error code and a flags value. Signed-off-by: Bart Van Assche Cc: Christoph Hellwig Cc: Hannes Reinecke Cc: Johannes Thumshirn Signed-off-by: Martin K. Petersen Signed-off-by: Sasha Levin Signed-off-by: Greg Kroah-Hartman --- drivers/scsi/scsi_devinfo.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) --- a/drivers/scsi/scsi_devinfo.c +++ b/drivers/scsi/scsi_devinfo.c @@ -589,17 +589,12 @@ int scsi_get_device_flags_keyed(struct s int key) { struct scsi_dev_info_list *devinfo; - int err; devinfo = scsi_dev_info_list_find(vendor, model, key); if (!IS_ERR(devinfo)) return devinfo->flags; - err = PTR_ERR(devinfo); - if (err != -ENOENT) - return err; - - /* nothing found, return nothing */ + /* key or device not found: return nothing */ if (key != SCSI_DEVINFO_GLOBAL) return 0;