diff options
| author |   Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> | 2016-12-28 09:26:35 -0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-12-28 14:06:31 -0500 |
| commit | 509e7a311f0423bce6d3e1705ba0a470ffe770dc (patch) | |
| tree | 15945c7d21464acc29c5f727603aeafaee252e9c | |
| parent | sctp: remove return value from sctp_packet_init/config (diff) | |
| download | linux-dev-509e7a311f0423bce6d3e1705ba0a470ffe770dc.tar.xz linux-dev-509e7a311f0423bce6d3e1705ba0a470ffe770dc.zip | |
sctp: sctp_chunk_length_valid should return bool
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/sctp/sm_statefuns.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/net/sctp/sm_statefuns.c b/net/sctp/sm_statefuns.c index 9a223d5b2314..3382ef254e7b 100644 --- a/net/sctp/sm_statefuns.c +++ b/net/sctp/sm_statefuns.c @@ -160,23 +160,22 @@ static sctp_disposition_t __sctp_sf_do_9_1_abort(struct net *net, /* Small helper function that checks if the chunk length * is of the appropriate length. The 'required_length' argument * is set to be the size of a specific chunk we are testing. - * Return Values: 1 = Valid length - * 0 = Invalid length + * Return Values: true = Valid length + * false = Invalid length * */ -static inline int -sctp_chunk_length_valid(struct sctp_chunk *chunk, - __u16 required_length) +static inline bool +sctp_chunk_length_valid(struct sctp_chunk *chunk, __u16 required_length) { __u16 chunk_length = ntohs(chunk->chunk_hdr->length); /* Previously already marked? */ if (unlikely(chunk->pdiscard)) - return 0; + return false; if (unlikely(chunk_length < required_length)) - return 0; + return false; - return 1; + return true; } /********************************************************** |