Re: [PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-09 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:30:18PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 11:11:35 +0100 > > This issue was detected by using the Coccinelle software. > > 1. Return directly if a call of the function "_ocrdma_alloc_pd"

Re: [PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-09 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:30:18PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 11:11:35 +0100 > > This issue was detected by using the Coccinelle software. > > 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. > > 2. Reduce the scope for

Re: [PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 7:00 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 11:11:35 +0100 > > This issue was detected by using the Coccinelle software.

Re: [PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 7:00 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 11:11:35 +0100 > > This issue was detected by using the Coccinelle software. > > 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. > > 2. Reduce

[PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 11:11:35 +0100 This issue was detected by using the Coccinelle software. 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. 2. Reduce the scope for the local variable "status" to one case of an if

[PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 11:11:35 +0100 This issue was detected by using the Coccinelle software. 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. 2. Reduce the scope for the local variable "status" to one case of an if statement. 3. Delete the jump