Skip to content

Commit

Permalink
[DCCP] CCID: Allow ccid_{init,exit} to be NULL
Browse files Browse the repository at this point in the history
Testing if the ccid being instantiated has these methods in
ccid_init().

Signed-off-by: Arnaldo Carvalho de Melo <acme@mandriva.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Arnaldo Carvalho de Melo authored and David S. Miller committed Mar 21, 2006
1 parent 02bcf28 commit 4114470
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 15 deletions.
5 changes: 1 addition & 4 deletions net/dccp/ccid.c
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,6 @@ int ccid_register(struct ccid *ccid)
{
int err;

if (ccid->ccid_init == NULL)
return -1;

ccids_write_lock();
err = -EEXIST;
if (ccids[ccid->ccid_id] == NULL) {
Expand Down Expand Up @@ -106,7 +103,7 @@ struct ccid *ccid_init(unsigned char id, struct sock *sk)
if (!try_module_get(ccid->ccid_owner))
goto out_err;

if (ccid->ccid_init(sk) != 0)
if (ccid->ccid_init != NULL && ccid->ccid_init(sk) != 0)
goto out_module_put;
out:
ccids_read_unlock();
Expand Down
11 changes: 0 additions & 11 deletions net/dccp/ccids/ccid3.c
Original file line number Diff line number Diff line change
Expand Up @@ -76,15 +76,6 @@ static struct dccp_tx_hist *ccid3_tx_hist;
static struct dccp_rx_hist *ccid3_rx_hist;
static struct dccp_li_hist *ccid3_li_hist;

static int ccid3_init(struct sock *sk)
{
return 0;
}

static void ccid3_exit(struct sock *sk)
{
}

/* TFRC sender states */
enum ccid3_hc_tx_states {
TFRC_SSTATE_NO_SENT = 1,
Expand Down Expand Up @@ -1182,8 +1173,6 @@ static struct ccid ccid3 = {
.ccid_id = 3,
.ccid_name = "ccid3",
.ccid_owner = THIS_MODULE,
.ccid_init = ccid3_init,
.ccid_exit = ccid3_exit,
.ccid_hc_tx_init = ccid3_hc_tx_init,
.ccid_hc_tx_exit = ccid3_hc_tx_exit,
.ccid_hc_tx_send_packet = ccid3_hc_tx_send_packet,
Expand Down

0 comments on commit 4114470

Please sign in to comment.