Re: [PATCH v3 4/4] cve: Add act_pedit page-cache corruption test

Cyril Hrubis <[email protected]>
Newsgroups gmane.linux.ltp
Message-ID <[email protected]>
Hi!
> +static void run(void)
> +{
> +	struct sockaddr_in addr;
> +	uint8_t readback[DATA_SIZE];
> +	off_t off = 0;
> +	int wr_fd;
> +
> +	wr_fd = SAFE_OPEN(TESTFILE, O_WRONLY | O_CREAT, 0444);
> +	SAFE_WRITE(SAFE_WRITE_ALL, wr_fd, original, DATA_SIZE);
> +	SAFE_CLOSE(wr_fd);
> +
> +	file_fd = SAFE_OPEN(TESTFILE, O_RDONLY);
> +
> +	/*
> +	 * Clean-slate clsact qdisc so no filter is active during the
> +	 * TCP handshake (the IHL-inflating key would corrupt SYN/ACK).
> +	 */
> +	tst_netdev_remove_qdisc(__FILE__, __LINE__, 0,
> +				"lo", AF_UNSPEC, TC_H_CLSACT,
> +				TC_H_MAKE(TC_H_CLSACT, 0), "clsact");

This is a bit ugly. I guess that we need to add a variant for the macro
that would set the strict to 0 into tst_netdevice.h possibly NETDEV_MAY_REMOVE_QDISC().


For the rest (including the preparatory patches):

Reviewed-by: Cyril Hrubis <[email protected]>

-- 
Cyril Hrubis
[email protected]

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.