Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp782265imm; Wed, 1 Aug 2018 05:20:02 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeI8lN3R/Erp4XUJ0q8NHtFG7O9Wy0EaopHPbSAIKuDTqNyEjrxCr0Q1e3QsQl7EEsVaTPZ X-Received: by 2002:a65:5284:: with SMTP id y4-v6mr23403877pgp.283.1533126002512; Wed, 01 Aug 2018 05:20:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533126002; cv=none; d=google.com; s=arc-20160816; b=VFWlpoetC4GumSQOV/wEdeBRLqG22t11I/8inphrUJ+/zLXUkQc/azP8vmRiBKUYd4 R6Ei5tdcRUhf3CNy7SKXMueGl7y3W3kbH2EPGFsm8WThuVoYK/rdo9dpJoC16C3nKq58 SX5YOpX20QQqZIx2s6+nOZXSJp5c8k7LAz6B0PtgbiCd16EiDKlUavL48CbFBm7qL5Gv nHMBs6ORgySRLirptoogtVcIPMzxZzW+jbYCgRCBRMdD3thwC/ulKpfatKXwNVbWFS8a kXcJag3v3oKolG1EAZF5saGlUBl6OsvfjJyEx3GGRZhIDxqHyuvW9thDsxB9GzrEOFzJ eMmA== 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 :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=oeVuGa4BGDxve00crzyYJPA2nKVzWBX8b6Pc0xGrofo=; b=EbN3r3lPjw6MabfMGEoRNyOo2xbWEsjlOM7FSaL2Gm1ZO4d2/71HAEaqi1fCCJf5Uv oVK7qSUQTXS+JfA09M1Wt813w3G527h+Eq7wF2J0vxRhR0P6ktjXWk7o1Qsx4qdCovKe O3V3w0+Qbjf/iXKWIa/lFGHxvaL1pKpdwdz5m3I49eYug1713AFJBDNYrJ8tBV07noCl BS+3LzLn+NqaBBHxTPO701JpPya8PLEb61Qtif5NjHG0j+HapW4s7QwaoAoiAoGLfYET mir1xxps2HW1mcsxHaO6co5TDHSAC/jkIDvsp+Uw4u9ThbVDPiGgqz8qFme+mwGU0joa n47g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b="MEs/yOoh"; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p5-v6si14822280pgl.516.2018.08.01.05.19.47; Wed, 01 Aug 2018 05:20:02 -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=@gmail.com header.s=20161025 header.b="MEs/yOoh"; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2388966AbeHAOEa (ORCPT + 99 others); Wed, 1 Aug 2018 10:04:30 -0400 Received: from mail-vk0-f65.google.com ([209.85.213.65]:36215 "EHLO mail-vk0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2387673AbeHAOEa (ORCPT ); Wed, 1 Aug 2018 10:04:30 -0400 Received: by mail-vk0-f65.google.com with SMTP id y9-v6so9162255vky.3 for ; Wed, 01 Aug 2018 05:18:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=oeVuGa4BGDxve00crzyYJPA2nKVzWBX8b6Pc0xGrofo=; b=MEs/yOoh7VyN2WsWKI/QoUJ95Fx1poMV9njd7jJh7Uk9lXdlm2P+01dNO5malXIwYu KMTxM2kcBB7wNSQf9jBXPhTzPc2vYu0ACrtxseBFlrpvpKtmVoCuVZRa218IyblxP60v yoNVcd6718jM0nBPCPIhwhO1vjZ/GFpe96ad4Ghhnev85xoWL2NNSX5xEQCW4dxTlKDw aXZcJRRvZ3k0AxwcCSUWEqFYQauag/F7FknruC29EbfXnHhzmYliuJWREE8HK/BDkrxa TYgov3ImlBZFfTkP9PZrFK5Jlk5mkw3SgEZrjIhhrw2y61/QHAH3GPVZAIxQT3esWpLk 7gaw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=oeVuGa4BGDxve00crzyYJPA2nKVzWBX8b6Pc0xGrofo=; b=QmqBqBTSSxDXmBmyci4Zf2hzkjxmsI9dNMnvEyoOz06nzrWm0vMY4ev6yUEgFthl/M yh5VTWN4FY/2GC+T0OWgKjH3w4sA3IUZIK0R2RNX2w4v2emKfGR/7Z3J/pUZkxtUVfME nOhBNcnvxOI3lSN4W7neP8uavJzBkOt7mDKDE/fqm8BJR/sRhZ77UQSQ3TpTNKCiyWTG WkBoxEydv2pWiceEY2zvbgS3qvfGO6kCGBZWQ7IXJPU5tv5g5MXUHF7+DbKRBsOXQs9o iUCFIEvlTV/fKYBXrPvNvzHQy7vQXAWfNwmsGPvwgiJRCgxjJVQ4oOPmXdY46t8B5wvJ qoKA== X-Gm-Message-State: AOUpUlERLB+PLRHkuW3G5gC1lqwQ99KHmk5w3ccWembs6bOBo2/r3fRW bD9a6Y3y0KQxdutcRuoCrSOgp7NXGZsYUwdStM0= X-Received: by 2002:a1f:b503:: with SMTP id e3-v6mr16690469vkf.97.1533125938622; Wed, 01 Aug 2018 05:18:58 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a67:2149:0:0:0:0:0 with HTTP; Wed, 1 Aug 2018 05:18:58 -0700 (PDT) In-Reply-To: <20180801115557.7086-1-vasilyev@ispras.ru> References: <20180801115557.7086-1-vasilyev@ispras.ru> From: Andy Shevchenko Date: Wed, 1 Aug 2018 15:18:58 +0300 Message-ID: Subject: Re: [PATCH v5] staging: rts5208: add error handling into rtsx_probe To: Anton Vasilyev Cc: Dan Carpenter , Greg Kroah-Hartman , Sinan Kaya , Johannes Thumshirn , Gaurav Pathak , Hannes Reinecke , devel@driverdev.osuosl.org, Linux Kernel Mailing List , ldv-project@linuxtesting.org 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 Wed, Aug 1, 2018 at 2:55 PM, Anton Vasilyev wrote: > If rtsx_probe() fails to allocate dev->chip, then release_everything() > will crash on uninitialized dev->cmnd_ready complete. > > Patch adds an error handling into rtsx_probe. > Found by Linux Driver Verification project (linuxtesting.org). Have you based your change on staging-next? Seems not. You need to rebase and resend. -- With Best Regards, Andy Shevchenko