Skip to content

Commit

Permalink
nf/dccp: merge errorpaths
Browse files Browse the repository at this point in the history
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Ilpo Järvinen authored and David S. Miller committed Dec 15, 2008
1 parent e780f1c commit 79f55f1
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions net/netfilter/xt_dccp.c
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,8 @@ dccp_find_option(u_int8_t option,
unsigned int optlen = dh->dccph_doff*4 - __dccp_hdr_len(dh);
unsigned int i;

if (dh->dccph_doff * 4 < __dccp_hdr_len(dh)) {
*hotdrop = true;
return false;
}
if (dh->dccph_doff * 4 < __dccp_hdr_len(dh))
goto invalid;

if (!optlen)
return false;
Expand All @@ -57,9 +55,7 @@ dccp_find_option(u_int8_t option,
op = skb_header_pointer(skb, protoff + optoff, optlen, dccp_optbuf);
if (op == NULL) {
/* If we don't have the whole header, drop packet. */
spin_unlock_bh(&dccp_buflock);
*hotdrop = true;
return false;
goto partial;
}

for (i = 0; i < optlen; ) {
Expand All @@ -76,6 +72,12 @@ dccp_find_option(u_int8_t option,

spin_unlock_bh(&dccp_buflock);
return false;

partial:
spin_unlock_bh(&dccp_buflock);
invalid:
*hotdrop = true;
return false;
}


Expand Down

0 comments on commit 79f55f1

Please sign in to comment.