lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20190528194051.GA7103@ip-172-31-44-144.us-west-2.compute.internal> Date: Tue, 28 May 2019 19:40:51 +0000 From: Alakesh Haloi <alakesh.haloi@...il.com> To: Song Liu <liu.song.a23@...il.com> Cc: Daniel Borkmann <daniel@...earbox.net>, Martin KaFai Lau <kafai@...com>, Song Liu <songliubraving@...com>, Yonghong Song <yhs@...com>, Stanislav Fomichev <sdf@...gle.com>, linux-kselftest@...r.kernel.org, Networking <netdev@...r.kernel.org>, bpf <bpf@...r.kernel.org>, open list <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] selftests: bpf: fix compiler warning On Tue, May 28, 2019 at 09:45:14AM -0700, Song Liu wrote: > On Thu, May 23, 2019 at 5:31 PM Alakesh Haloi <alakesh.haloi@...il.com> wrote: > > > > Add missing header file following compiler warning > > > > prog_tests/flow_dissector.c: In function ‘tx_tap’: > > prog_tests/flow_dissector.c:175:9: warning: implicit declaration of function ‘writev’; did you mean ‘write’? [-Wimplicit-function-declaration] > > return writev(fd, iov, ARRAY_SIZE(iov)); > > ^~~~~~ > > write > > > > Signed-off-by: Alakesh Haloi <alakesh.haloi@...il.com> > > The patch looks good. Please add a "Fixes" tag, so the fix > can be back ported properly. > > Also, please specify which tree the patch should be applied > with [PATCH bpf] or [PATCH bpf-next]. > > Thanks, > Song > Thanks for reviewing the patch. My apologies for not following the rules. I have sent an updated patch after adding Fixes: tag and modifying the subject to reflect that it is for bpf tree. The updated patch is here https://lkml.org/lkml/2019/5/28/904 Thanks Alakesh > > > > > --- > > tools/testing/selftests/bpf/prog_tests/flow_dissector.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/tools/testing/selftests/bpf/prog_tests/flow_dissector.c b/tools/testing/selftests/bpf/prog_tests/flow_dissector.c > > index fbd1d88a6095..c938283ac232 100644 > > --- a/tools/testing/selftests/bpf/prog_tests/flow_dissector.c > > +++ b/tools/testing/selftests/bpf/prog_tests/flow_dissector.c > > @@ -3,6 +3,7 @@ > > #include <error.h> > > #include <linux/if.h> > > #include <linux/if_tun.h> > > +#include <sys/uio.h> > > > > #define CHECK_FLOW_KEYS(desc, got, expected) \ > > CHECK_ATTR(memcmp(&got, &expected, sizeof(got)) != 0, \ > > -- > > 2.17.1 > >
Powered by blists - more mailing lists