Re: [Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-06 Thread Boris Ostrovsky
On 12/05/2017 01:29 PM, Stefano Stabellini wrote: > On Tue, 5 Dec 2017, Dan Carpenter wrote: >> bedata->ref can't be less than zero because it's unsigned. This affects >> certain error paths in probe. We first set ->ref = -1 and then we set >> it to a valid value later. >> >> Fixes: 219681909913

Re: [Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-05 Thread Stefano Stabellini
On Tue, 5 Dec 2017, Dan Carpenter wrote: > bedata->ref can't be less than zero because it's unsigned. This affects > certain error paths in probe. We first set ->ref = -1 and then we set > it to a valid value later. > > Fixes: 219681909913 ("xen/pvcalls: connect to the backend") > Signed-off-by:

Re: [Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-05 Thread Juergen Gross
On 05/12/17 15:38, Dan Carpenter wrote: > bedata->ref can't be less than zero because it's unsigned. This affects > certain error paths in probe. We first set ->ref = -1 and then we set > it to a valid value later. > > Fixes: 219681909913 ("xen/pvcalls: connect to the backend") > Signed-off-by:

[Xen-devel] [PATCH 2/2] xen/pvcalls: Fix a check in pvcalls_front_remove()

2017-12-05 Thread Dan Carpenter
bedata->ref can't be less than zero because it's unsigned. This affects certain error paths in probe. We first set ->ref = -1 and then we set it to a valid value later. Fixes: 219681909913 ("xen/pvcalls: connect to the backend") Signed-off-by: Dan Carpenter diff --git a/drivers/xen/pvcalls-fro