diff options
| author | Markus Pargmann <mpa@pengutronix.de> | 2014-12-26 12:41:23 +0100 |
|---|---|---|
| committer | Antonio Quartulli <antonio@meshcoding.com> | 2015-05-29 10:13:36 +0200 |
| commit | 16b9ce83fb850602794a3bc534693362408a5408 (patch) | |
| tree | 690d3c19f757404b86495ecb6f266cfeb9c42a89 /net/batman-adv/main.c | |
| parent | 9bb218828c8f4fa6587af93e248903c96ce469d0 (diff) | |
| download | op-kernel-dev-16b9ce83fb850602794a3bc534693362408a5408.zip op-kernel-dev-16b9ce83fb850602794a3bc534693362408a5408.tar.gz | |
batman-adv: tvlv realloc, move error handling into if block
Instead of hiding the normal function flow inside an if block, we should
just put the error handling into the if block.
Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch>
Diffstat (limited to 'net/batman-adv/main.c')
| -rw-r--r-- | net/batman-adv/main.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/net/batman-adv/main.c b/net/batman-adv/main.c index ca63d48..fd9333d 100644 --- a/net/batman-adv/main.c +++ b/net/batman-adv/main.c @@ -819,15 +819,15 @@ static bool batadv_tvlv_realloc_packet_buff(unsigned char **packet_buff, new_buff = kmalloc(min_packet_len + additional_packet_len, GFP_ATOMIC); /* keep old buffer if kmalloc should fail */ - if (new_buff) { - memcpy(new_buff, *packet_buff, min_packet_len); - kfree(*packet_buff); - *packet_buff = new_buff; - *packet_buff_len = min_packet_len + additional_packet_len; - return true; - } + if (!new_buff) + return false; + + memcpy(new_buff, *packet_buff, min_packet_len); + kfree(*packet_buff); + *packet_buff = new_buff; + *packet_buff_len = min_packet_len + additional_packet_len; - return false; + return true; } /** |
