Message ID | 20180517233105.8243-6-bsingharora@gmail.com |
---|---|
State | Accepted |
Headers | show
Return-Path: <skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org> X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 40n6zv5bnvz9s1w for <incoming@patchwork.ozlabs.org>; Fri, 18 May 2018 09:32:59 +1000 (AEST) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="M3lP6IXU"; dkim-atps=neutral Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 40n6zv3r2FzF28D for <incoming@patchwork.ozlabs.org>; Fri, 18 May 2018 09:32:59 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="M3lP6IXU"; dkim-atps=neutral X-Original-To: skiboot@lists.ozlabs.org Delivered-To: skiboot@lists.ozlabs.org Authentication-Results: lists.ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=gmail.com (client-ip=2607:f8b0:400e:c00::229; helo=mail-pf0-x229.google.com; envelope-from=bsingharora@gmail.com; receiver=<UNKNOWN>) Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="M3lP6IXU"; dkim-atps=neutral Received: from mail-pf0-x229.google.com (mail-pf0-x229.google.com [IPv6:2607:f8b0:400e:c00::229]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 40n6yD5N2WzF294 for <skiboot@lists.ozlabs.org>; Fri, 18 May 2018 09:31:32 +1000 (AEST) Received: by mail-pf0-x229.google.com with SMTP id e9-v6so2822448pfi.4 for <skiboot@lists.ozlabs.org>; Thu, 17 May 2018 16:31:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=ZOXRB3814wl026/P58gcHw9hS1AQSDVXkCsnadfKcZY=; b=M3lP6IXUaI2HH9clFqjy5tekCWAPZIIruLu41/XYeUg80KW4Mnatxcw+8iKV9OINPx cRr1spg0xqjl85C9dSqSmt3IKBrImLdaqPHr3iSSsJeuVH1x0yqS2PY4PRcAWB2Xj1XW fn/5OtDkdLn+XUhdxWVrX2QRNnXu/KYn11h3tzpcSUvR4hHBvuZJaILMZ47+3U60qdpT CqdFIM4Y2dH8bq0CzcnI1qZgI2Bc6QggELCOjkjzeFfTwiVB3rmk4Zy3TUANxMWk/XHU OfSeH4EZSZco9o5xNv5z8+gz/hc0U7fmLIleTrcwEmQ9hqW/z+4QPnenVctAU807Cek3 YvCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=ZOXRB3814wl026/P58gcHw9hS1AQSDVXkCsnadfKcZY=; b=C4qFMCt8UcNtx70H/GrBJREHfrhhWBX6XCTZNIomXm4UaZqXOYdfp/vwH8f4+HJVff pL7ezfiqhQrLkSaQrWoyZWryDNdbkr2nFGqaOYDLy+I2EiNf09SFWCWFu/qrFXebTOIo MQhVgfbp9eqtP6mfQZT5NZz+pLqdHy4+xrNw9BPxffBs82mE+JP4wtYj7XJQE2b2RTX9 2UfwmK5AGOSm09biwqF28Tab7BHMdquAyZF/vPfFSNrR7yPtsIPtOBRaq5EQp/21HLzN Qfr9uDAF45L9iK7QQjVxAzOpEbP6ys7TqxwF2+KvZWzbmZKHks0dQ+8x7WUesjoZt/Gv 1wYw== X-Gm-Message-State: ALKqPwdCvNksZqQg1ajCX7YyTifpFBcnVbuTA1Za+/sHRw+XfEP/MB8/ af/URjiVYjQ4V23wEneZlAFxf6r4 X-Google-Smtp-Source: AB8JxZpCParJio6M1sv7L7tnq9XsyDXlXhC3tn3PMMjTZlV6d/24PyKRl4dMuvFownEDragiONoqiA== X-Received: by 2002:a62:6402:: with SMTP id y2-v6mr6816889pfb.71.1526599890233; Thu, 17 May 2018 16:31:30 -0700 (PDT) Received: from localhost.localdomain (14-202-194-140.static.tpgi.com.au. [14.202.194.140]) by smtp.gmail.com with ESMTPSA id q76-v6sm12805990pfi.139.2018.05.17.16.31.27 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 17 May 2018 16:31:29 -0700 (PDT) From: Balbir singh <bsingharora@gmail.com> To: skiboot@lists.ozlabs.org Date: Fri, 18 May 2018 09:31:04 +1000 Message-Id: <20180517233105.8243-6-bsingharora@gmail.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180517233105.8243-1-bsingharora@gmail.com> References: <20180517233105.8243-1-bsingharora@gmail.com> Subject: [Skiboot] [PATCH 5/6] libflash/blocklevel_write: Fix missing error handling X-BeenThere: skiboot@lists.ozlabs.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: Mailing list for skiboot development <skiboot.lists.ozlabs.org> List-Unsubscribe: <https://lists.ozlabs.org/options/skiboot>, <mailto:skiboot-request@lists.ozlabs.org?subject=unsubscribe> List-Archive: <http://lists.ozlabs.org/pipermail/skiboot/> List-Post: <mailto:skiboot@lists.ozlabs.org> List-Help: <mailto:skiboot-request@lists.ozlabs.org?subject=help> List-Subscribe: <https://lists.ozlabs.org/listinfo/skiboot>, <mailto:skiboot-request@lists.ozlabs.org?subject=subscribe> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org Sender: "Skiboot" <skiboot-bounces+incoming=patchwork.ozlabs.org@lists.ozlabs.org> |
Series |
scan build fixes
|
expand
|
diff --git a/libflash/blocklevel.c b/libflash/blocklevel.c index f3fad69e..172f8fcc 100644 --- a/libflash/blocklevel.c +++ b/libflash/blocklevel.c @@ -263,6 +263,7 @@ int blocklevel_write(struct blocklevel_device *bl, uint64_t pos, const void *buf if (rc) { errno = EBADF; rc = FLASH_ERR_ECC_INVALID; + goto out; } /* @@ -274,6 +275,7 @@ int blocklevel_write(struct blocklevel_device *bl, uint64_t pos, const void *buf if (rc) { errno = EBADF; rc = FLASH_ERR_ECC_INVALID; + goto out; } if (memcpy_to_ecc_unaligned(buffer, buf, len, ecc_diff)) {
Caught by scan-build, we seem to trap the errors in rc, but not take any recovery action during blocklevel_write. Signed-off-by: Balbir singh <bsingharora@gmail.com> --- libflash/blocklevel.c | 2 ++ 1 file changed, 2 insertions(+)