From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751982AbdIQWfE (ORCPT ); Sun, 17 Sep 2017 18:35:04 -0400 Received: from smtp07.smtpout.orange.fr ([80.12.242.129]:17906 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751533AbdIQWfC (ORCPT ); Sun, 17 Sep 2017 18:35:02 -0400 X-ME-Helo: localhost.localdomain X-ME-Auth: Y2hyaXN0b3BoZS5qYWlsbGV0QHdhbmFkb28uZnI= X-ME-Date: Mon, 18 Sep 2017 00:35:01 +0200 X-ME-IP: 86.196.182.67 From: Christophe JAILLET To: mporter@kernel.crashing.org, alexandre.bounine@idt.com, lstoakes@gmail.com, akpm@linux-foundation.org, jesper.nilsson@axis.com, christian.koenig@amd.com, logang@deltatee.com Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET Subject: [PATCH 0/2] rapidio: Two fixes for 'rio_dma_transfer()' Date: Mon, 18 Sep 2017 00:33:04 +0200 Message-Id: X-Mailer: git-send-email 2.11.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org These 2 patches are really close and are both related to error handling in 'rio_dma_transfer()'. The first one fixes some resources leak in case of error. The 2nd one fixes a test so that the corresponding error can be handled correctly. I've splitted this into 2 paches because the first one looks trivial to me. But the 2nd one should require more attention, IMO. The existing error handling was dead code because 'dma_map_sg()' can not return -EFAULT. So this error handling code has never been executed/tested and I can't test it myself. Christophe JAILLET (2): rapidio: Fix resources leak in error handling path in 'rio_dma_transfer()' rapidio: Fix an error handling in 'rio_dma_transfer()' drivers/rapidio/devices/rio_mport_cdev.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) -- 2.11.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christophe JAILLET Date: Sun, 17 Sep 2017 22:33:04 +0000 Subject: [PATCH 0/2] rapidio: Two fixes for 'rio_dma_transfer()' Message-Id: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: mporter@kernel.crashing.org, alexandre.bounine@idt.com, lstoakes@gmail.com, akpm@linux-foundation.org, jesper.nilsson@axis.com, christian.koenig@amd.com, logang@deltatee.com Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET These 2 patches are really close and are both related to error handling in 'rio_dma_transfer()'. The first one fixes some resources leak in case of error. The 2nd one fixes a test so that the corresponding error can be handled correctly. I've splitted this into 2 paches because the first one looks trivial to me. But the 2nd one should require more attention, IMO. The existing error handling was dead code because 'dma_map_sg()' can not return -EFAULT. So this error handling code has never been executed/tested and I can't test it myself. Christophe JAILLET (2): rapidio: Fix resources leak in error handling path in 'rio_dma_transfer()' rapidio: Fix an error handling in 'rio_dma_transfer()' drivers/rapidio/devices/rio_mport_cdev.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) -- 2.11.0