netfilter: nfnetlink_log: Change setter functions to be void
authorRosen, Rami <rami.rosen@intel.com>
Tue, 8 Dec 2015 12:09:24 +0000 (07:09 -0500)
committerPablo Neira Ayuso <pablo@netfilter.org>
Wed, 9 Dec 2015 13:52:56 +0000 (14:52 +0100)
Change return type of nfulnl_set_timeout() and nfulnl_set_qthresh() to
be void.

This patch changes the return type of the static methods
nfulnl_set_timeout() and nfulnl_set_qthresh() to be void, as there is no
justification and no need for these methods to return int.

Signed-off-by: Rami Rosen <rami.rosen@intel.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nfnetlink_log.c

index dea467647c90d6641d812fa9e2f9babbf75c1f1e..70b6bd3b781e46bb94e05c0a870c5900b0d4fd74 100644 (file)
@@ -293,24 +293,20 @@ nfulnl_set_nlbufsiz(struct nfulnl_instance *inst, u_int32_t nlbufsiz)
        return status;
 }
 
-static int
+static void
 nfulnl_set_timeout(struct nfulnl_instance *inst, u_int32_t timeout)
 {
        spin_lock_bh(&inst->lock);
        inst->flushtimeout = timeout;
        spin_unlock_bh(&inst->lock);
-
-       return 0;
 }
 
-static int
+static void
 nfulnl_set_qthresh(struct nfulnl_instance *inst, u_int32_t qthresh)
 {
        spin_lock_bh(&inst->lock);
        inst->qthreshold = qthresh;
        spin_unlock_bh(&inst->lock);
-
-       return 0;
 }
 
 static int