[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20190125214320.17685-1-johannes@sipsolutions.net>
Date: Fri, 25 Jan 2019 22:43:16 +0100
From: Johannes Berg <johannes@...solutions.net>
To: netdev@...r.kernel.org
Cc: Al Viro <viro@...iv.linux.org.uk>,
Robert O'Callahan <robert@...llahan.org>
Subject: [PATCH net 0/4] various compat ioctl fixes
Back a long time ago, I already fixed a few of these by passing
the size of the struct ifreq to do_sock_ioctl(). However, Robert
found more cases, and now it won't be as simple because we'd have
to pass that down all the way to e.g. bond_do_ioctl() which isn't
really feasible.
Therefore, restore the old code.
While looking at why SIOCGIFNAME was broken, I realized that Al
had removed that case - which had been handled in an explicit
separate function - as well, and looking through his work at the
time I saw that bond ioctls were also affected by the erroneous
removal.
I've restored SIOCGIFNAME and bond ioctls by going through the
(now renamed) dev_ifsioc() instead of reintroducing their own
helper functions, which I hope is correct but have only tested
with SIOCGIFNAME.
johannes
Powered by blists - more mailing lists