dissector: ip_authentication_hdr: Remove trailing whitespaces

Remove two trailing whitespaces on emtpy lines.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
dissector: ip_ipv6_mobility_hdr: NULL check before dereference

Check return value of pkt_pull before dereferencing it. This was
discovered by the coverity scanner.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
flowtop: Use size of destination buffer in memcpy

The coverity scanner found the macro used as argument to sizeof
suspicious. Use sizeof the destination buffer instead.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
dissector: lldp: NULL check before dereference

Check return value of pkt_pull before dereferencing it (even though we
check the packet length before and pkt_pull _should_ never return NULL).
This was discovered by the coverity scanner.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
geoip: Don't leak file/socket descriptor in error case

In certain error cases, the file/socket descriptors are not closed
in geoip_get_database(). This was discovered by the coverity scanner.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
xmalloc: Make xfree a two part macro/inline

This way the coverity scanner will no longer complain about dereference
before NULL checks right before xfree().

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
dissector: igmp: Fix erroneous null pointer check

We need to break if we get NULL from pkt_pull() but the actual check was
inverted. This was discovered by the coverity scanner.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>