Re: [PATCH] switchtec: Fix an error handling
On 12/04/17 11:27 AM, Bjorn Helgaas wrote: > On Tue, Apr 11, 2017 at 06:32:04AM +0200, Christophe JAILLET wrote: >> 'stuser_create' returns an error pointer in case of error, not NULL. >> So test its return value with IS_ERR. >> >> Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") >> >> Signed-off-by: Christophe JAILLET> > Since this hasn't been merged to Linus' tree yet, I folded the fix > into the original commit on pci/switchtec and added attribution to > Christophe. Thanks Bjorn. Logan
Re: [PATCH] switchtec: Fix an error handling
On 12/04/17 11:27 AM, Bjorn Helgaas wrote: > On Tue, Apr 11, 2017 at 06:32:04AM +0200, Christophe JAILLET wrote: >> 'stuser_create' returns an error pointer in case of error, not NULL. >> So test its return value with IS_ERR. >> >> Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") >> >> Signed-off-by: Christophe JAILLET > > Since this hasn't been merged to Linus' tree yet, I folded the fix > into the original commit on pci/switchtec and added attribution to > Christophe. Thanks Bjorn. Logan
Re: [PATCH] switchtec: Fix an error handling
On Tue, Apr 11, 2017 at 06:32:04AM +0200, Christophe JAILLET wrote: > 'stuser_create' returns an error pointer in case of error, not NULL. > So test its return value with IS_ERR. > > Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") > > Signed-off-by: Christophe JAILLETSince this hasn't been merged to Linus' tree yet, I folded the fix into the original commit on pci/switchtec and added attribution to Christophe. > --- > drivers/pci/switch/switchtec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index fcde98161d9a..cc6e085008fb 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct > file *filp) > stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); > > stuser = stuser_create(stdev); > - if (!stuser) > + if (IS_ERR(stuser)) > return PTR_ERR(stuser); > > filp->private_data = stuser; > -- > 2.11.0 >
Re: [PATCH] switchtec: Fix an error handling
On Tue, Apr 11, 2017 at 06:32:04AM +0200, Christophe JAILLET wrote: > 'stuser_create' returns an error pointer in case of error, not NULL. > So test its return value with IS_ERR. > > Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") > > Signed-off-by: Christophe JAILLET Since this hasn't been merged to Linus' tree yet, I folded the fix into the original commit on pci/switchtec and added attribution to Christophe. > --- > drivers/pci/switch/switchtec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index fcde98161d9a..cc6e085008fb 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct > file *filp) > stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); > > stuser = stuser_create(stdev); > - if (!stuser) > + if (IS_ERR(stuser)) > return PTR_ERR(stuser); > > filp->private_data = stuser; > -- > 2.11.0 >
Re: [PATCH] switchtec: Fix an error handling
Nice catch. Thanks. Reviewed-by: Logan GunthorpeLogan On 10/04/17 10:32 PM, Christophe JAILLET wrote: > 'stuser_create' returns an error pointer in case of error, not NULL. > So test its return value with IS_ERR. > > Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") > > Signed-off-by: Christophe JAILLET > --- > drivers/pci/switch/switchtec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index fcde98161d9a..cc6e085008fb 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct > file *filp) > stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); > > stuser = stuser_create(stdev); > - if (!stuser) > + if (IS_ERR(stuser)) > return PTR_ERR(stuser); > > filp->private_data = stuser; >
Re: [PATCH] switchtec: Fix an error handling
Nice catch. Thanks. Reviewed-by: Logan Gunthorpe Logan On 10/04/17 10:32 PM, Christophe JAILLET wrote: > 'stuser_create' returns an error pointer in case of error, not NULL. > So test its return value with IS_ERR. > > Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") > > Signed-off-by: Christophe JAILLET > --- > drivers/pci/switch/switchtec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c > index fcde98161d9a..cc6e085008fb 100644 > --- a/drivers/pci/switch/switchtec.c > +++ b/drivers/pci/switch/switchtec.c > @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct > file *filp) > stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); > > stuser = stuser_create(stdev); > - if (!stuser) > + if (IS_ERR(stuser)) > return PTR_ERR(stuser); > > filp->private_data = stuser; >
[PATCH] switchtec: Fix an error handling
'stuser_create' returns an error pointer in case of error, not NULL. So test its return value with IS_ERR. Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") Signed-off-by: Christophe JAILLET--- drivers/pci/switch/switchtec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c index fcde98161d9a..cc6e085008fb 100644 --- a/drivers/pci/switch/switchtec.c +++ b/drivers/pci/switch/switchtec.c @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct file *filp) stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); stuser = stuser_create(stdev); - if (!stuser) + if (IS_ERR(stuser)) return PTR_ERR(stuser); filp->private_data = stuser; -- 2.11.0
[PATCH] switchtec: Fix an error handling
'stuser_create' returns an error pointer in case of error, not NULL. So test its return value with IS_ERR. Fixes: 74004262f329 ("MicroSemi Switchtec management interface driver") Signed-off-by: Christophe JAILLET --- drivers/pci/switch/switchtec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c index fcde98161d9a..cc6e085008fb 100644 --- a/drivers/pci/switch/switchtec.c +++ b/drivers/pci/switch/switchtec.c @@ -608,7 +608,7 @@ static int switchtec_dev_open(struct inode *inode, struct file *filp) stdev = container_of(inode->i_cdev, struct switchtec_dev, cdev); stuser = stuser_create(stdev); - if (!stuser) + if (IS_ERR(stuser)) return PTR_ERR(stuser); filp->private_data = stuser; -- 2.11.0