HSI: nokia-modem: fix error return code
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sat, 22 Nov 2014 14:39:16 +0000 (15:39 +0100)
committerSebastian Reichel <sre@kernel.org>
Sat, 28 Feb 2015 19:14:13 +0000 (20:14 +0100)
Return a negative error code on failure.

A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
identifier ret; expression e1,e2;
@@
(
if (\(ret < 0\|ret != 0\))
 { ... return ret; }
|
ret = 0
)
... when != ret = e1
    when != &ret
*if(...)
{
  ... when != ret = e2
      when forall
 return ret;
}
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
drivers/hsi/clients/nokia-modem.c

index eb4dc63..9be4867 100644 (file)
@@ -200,6 +200,7 @@ static int nokia_modem_probe(struct device *dev)
        modem->ssi_protocol = hsi_new_client(port, &ssip);
        if (!modem->ssi_protocol) {
                dev_err(dev, "Could not register ssi-protocol device\n");
+               err = -ENOMEM;
                goto error2;
        }