Bluetooth: Make the function sco_conn_del have a return type of void
authorNicholas Krause <xerofoify@gmail.com>
Wed, 19 Aug 2015 01:23:01 +0000 (21:23 -0400)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 28 Aug 2015 19:00:37 +0000 (21:00 +0200)
This makes the function sco_conn_del have a return type of void now
due to this function always running successfully and thus never
needing to signal its caller when a non recoverable internal failure
occurs by returning a error code to its respective caller.

Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/sco.c

index 688a040..f315c8d 100644 (file)
@@ -154,13 +154,13 @@ static void sco_chan_del(struct sock *sk, int err)
        sock_set_flag(sk, SOCK_ZAPPED);
 }
 
-static int sco_conn_del(struct hci_conn *hcon, int err)
+static void sco_conn_del(struct hci_conn *hcon, int err)
 {
        struct sco_conn *conn = hcon->sco_data;
        struct sock *sk;
 
        if (!conn)
-               return 0;
+               return;
 
        BT_DBG("hcon %p conn %p, err %d", hcon, conn, err);
 
@@ -179,7 +179,6 @@ static int sco_conn_del(struct hci_conn *hcon, int err)
 
        hcon->sco_data = NULL;
        kfree(conn);
-       return 0;
 }
 
 static void __sco_chan_add(struct sco_conn *conn, struct sock *sk, struct sock *parent)