[LTP] [PATCH 1/1] syscalls, lib: Use SAFE_{BIND, LISTEN, SOCKET} macros
Petr Vorel
pvorel@suse.cz
Sat Sep 30 01:15:32 CEST 2017
> instead of tst_brk{,m}(TBROK, ...)
> Patch was made thanks to Coccinelle.
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
Please ignore second sign.
<snip>
> diff --git a/lib/tst_net.c b/lib/tst_net.c
> index f842e94a6..326b92e6f 100644
> --- a/lib/tst_net.c
> +++ b/lib/tst_net.c
> @@ -25,6 +25,7 @@
> #include <netinet/in.h>
> #include "test.h"
> +#include "safe_macros.h"
> unsigned short tst_get_unused_port(void (cleanup_fn)(void),
> unsigned short family, int type)
> @@ -57,16 +58,9 @@ unsigned short tst_get_unused_port(void (cleanup_fn)(void),
> return -1;
> }
> - sock = socket(addr->sa_family, type, 0);
> - if (sock < 0) {
> - tst_brkm(TBROK | TERRNO, cleanup_fn, "socket failed");
> - return -1;
> - }
> + sock = SAFE_SOCKET(cleanup_fn, addr->sa_family, type, 0);
> - if (bind(sock, addr, slen) < 0) {
> - tst_brkm(TBROK | TERRNO, cleanup_fn, "bind failed");
> - return -1;
> - }
> + SAFE_BIND(cleanup_fn, sock, addr, slen);
I suppose tst_brkm(TBROK, ...) always exit testing, so we can have SAFE_* macros even here
in lib.
Kind regards,
Petr
More information about the ltp
mailing list