Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp323802imm; Sat, 14 Jul 2018 01:17:15 -0700 (PDT) X-Google-Smtp-Source: AAOMgpd5depaJ9DpRiigO/7JwosYAES9d+6SPHaspDOBlrFUuQrVQ7IyynfrbbbVOyH5zK0xwniO X-Received: by 2002:a63:2ac4:: with SMTP id q187-v6mr2495656pgq.333.1531556235057; Sat, 14 Jul 2018 01:17:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1531556235; cv=none; d=google.com; s=arc-20160816; b=HJFhjReJQNcB96mVqzQnXU98tTJdPXzGUcZBVlKJBfe87uXJHO4MIwi2utnXl3p1p1 XuB1mqbPZKbIPbI0T1R8rRxmZnMfLeIukq2BAVCqBEK+9wV5iLVqzXoC6m+k5L9DcJmL Qz0pTvxeFGDK2VmoMFy4gKI6ququzzNbZFVBDSu7lYdqDj4+j4bdzEVgyp5GSvzdEiaH QkHv+6i4SU4rCaE1HqlAcjoqSVjaMEjL7MrBGe6oRl7r+BLLXV9vVBvQ1avGqgLsAf+p SrWI2k3QqtUIsMRktRQwt43cbpSdQim2XVk49vCkrvAG9UeLTGKmF4cOJ8IeZzU0hwSR tPEw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature :arc-authentication-results; bh=iflpxDqFo3iHd8++S1H7X2GPCTJMUhlfuvFXqcPtMMU=; b=MAap8pgrUZ7LkSudmsE535+IpQm57x4mi+9Xb9/INUKC9czTaqIOS/V5hLbNgE7/3T p14P1pwqlHU7dlDVEIVxxpgBVa+LYJFW46ir8Cq41SipmniEGdzTEg+o0n9Y0LjVe7wO uAS1ibSZKnNcmRXA6zt1WXmLYdY4H6RwwaNLb6mE4thFNZPJ/EjtfCPIkKNPTrWKIA07 q2WrzAIjHz8YWFSLNMrsSg+xniHwgRr5rbaYZZoFggt3bIWdof8w/U8iotZV1zfpurEF ZeeQEcF1/g/Tg8NcUEWFdWuU+TExtYRrNpY5ly0pFuEP9yp373IrAZS+UItVkvyyMDXf Jh4w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=R0uzFI7P; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g80-v6si27743140pfk.53.2018.07.14.01.17.00; Sat, 14 Jul 2018 01:17:15 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=R0uzFI7P; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727091AbeGNIee (ORCPT + 99 others); Sat, 14 Jul 2018 04:34:34 -0400 Received: from mail-ed1-f67.google.com ([209.85.208.67]:34422 "EHLO mail-ed1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726098AbeGNIee (ORCPT ); Sat, 14 Jul 2018 04:34:34 -0400 Received: by mail-ed1-f67.google.com with SMTP id d3-v6so26303321edi.1 for ; Sat, 14 Jul 2018 01:16:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=iflpxDqFo3iHd8++S1H7X2GPCTJMUhlfuvFXqcPtMMU=; b=R0uzFI7PJhtvaZ/3cvagkGZ007gSIhV8iX0nNc7Y4bLpqXFslFL0Aiucfg+gcKOSMM BcTm2Q6o/iN4IunOFn3in/d6oED+7Wl2CjozHvrzJfLFCEnRpmThDKKhaEa3sY4nY6Fl BwrGy1yYWWkd2zDyOURHk0hB6zBkyI1nM61uU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=iflpxDqFo3iHd8++S1H7X2GPCTJMUhlfuvFXqcPtMMU=; b=bpu0fVfKvtFtAFuauj4fRhvKmW0zmVPqDuJeqfVav+/ulybl2AlYF4YybBp5TtbCBK qGbGyQycIv3UPyd5jFpJSLZ0pKfobJ7mtu1RJkPM1j/y0ota/TGsJ0sf2MlWKXUnkS/x EXlR/gYlob8VtVFWY1O1TJaAPSPfvkIenIXHFTvI5+UhlDSgsfHLOIQESm5z8M7euVOQ V2J+yaoHX05kKos/Qa1uWwVacr1Zoc3UVggzoEsg8Go2P7eim8CeUm/BOYGzJKZ9FFRU bJRj33xQhoXA648n/uZjwujaXDmWwOMR6w0IChJJ/rXL9mUBf+APGy4xTou7QyqKR5i4 ++pg== X-Gm-Message-State: AOUpUlGzw0DDMlOM/isabdJXVEjqvU6irut263uhrTFLEQjn3XSMn4aJ NouitO8cLpOwJvXxGjCQ6tjRRn0Mfe+dXg== X-Received: by 2002:a50:bdc7:: with SMTP id z7-v6mr10221541edh.84.1531556177282; Sat, 14 Jul 2018 01:16:17 -0700 (PDT) Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com. [209.85.221.50]) by smtp.gmail.com with ESMTPSA id g14-v6sm15565534edm.25.2018.07.14.01.16.15 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 14 Jul 2018 01:16:16 -0700 (PDT) Received: by mail-wr1-f50.google.com with SMTP id a3-v6so17913615wrt.2 for ; Sat, 14 Jul 2018 01:16:15 -0700 (PDT) X-Received: by 2002:adf:8325:: with SMTP id 34-v6mr6852030wrd.67.1531556175198; Sat, 14 Jul 2018 01:16:15 -0700 (PDT) MIME-Version: 1.0 References: <20180714055816.223754-1-toddpoynor@gmail.com> <20180714055816.223754-15-toddpoynor@gmail.com> In-Reply-To: <20180714055816.223754-15-toddpoynor@gmail.com> From: Dmitry Torokhov Date: Sat, 14 Jul 2018 11:16:03 +0300 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 14/18] staging: gasket: fix class create bug handling To: toddpoynor@gmail.com Cc: rspringer@google.com, jnjoseph@google.com, benchan@chromium.org, Greg Kroah-Hartman , frankhu@chromium.org, Simon Que , Guenter Roeck , devel@driverdev.osuosl.org, lkml , toddpoynor@google.com Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jul 14, 2018 at 8:59 AM Todd Poynor wrote: > > From: Todd Poynor > > class_create() never returns NULL, and this driver should never return > PTR_ERR(NULL) anyway. > > Reported-by: Dmitry Torokhov > Signed-off-by: Zhongze Hu > Signed-off-by: Todd Poynor Reviewed-by: Dmitry Torokhov > --- > drivers/staging/gasket/gasket_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/gasket/gasket_core.c b/drivers/staging/gasket/gasket_core.c > index f7d8f66e8746..0ef37667e0f1 100644 > --- a/drivers/staging/gasket/gasket_core.c > +++ b/drivers/staging/gasket/gasket_core.c > @@ -341,7 +341,7 @@ int gasket_register_device(const struct gasket_driver_desc *driver_desc) > internal->class = > class_create(driver_desc->module, driver_desc->name); > > - if (IS_ERR_OR_NULL(internal->class)) { > + if (IS_ERR(internal->class)) { > gasket_nodev_error("Cannot register %s class [ret=%ld]", > driver_desc->name, PTR_ERR(internal->class)); > ret = PTR_ERR(internal->class); > -- > 2.18.0.203.gfac676dfb9-goog >