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
| ||
|
Date: Sat, 2 Dec 2017 08:41:55 +0100 From: Johannes Berg <johannes@...solutions.net> To: netdev@...r.kernel.org Cc: Jouni Malinen <j@...fi>, Rasmus Villemoes <linux@...musvillemoes.dk>, Johannes Berg <johannes.berg@...el.com> Subject: [PATCH net] Revert "net: core: maybe return -EEXIST in __dev_alloc_name" From: Johannes Berg <johannes.berg@...el.com> This reverts commit d6f295e9def0; some userspace (in the case we noticed it's wpa_supplicant), is relying on the current error code to determine that a fixed name interface already exists. Reported-by: Jouni Malinen <j@...fi> Signed-off-by: Johannes Berg <johannes.berg@...el.com> --- net/core/dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/core/dev.c b/net/core/dev.c index 07ed21d64f92..f47e96b62308 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -1106,7 +1106,7 @@ static int __dev_alloc_name(struct net *net, const char *name, char *buf) * when the name is long and there isn't enough space left * for the digits, or if all bits are used. */ - return p ? -ENFILE : -EEXIST; + return -ENFILE; } static int dev_alloc_name_ns(struct net *net, -- 2.14.2
Powered by blists - more mailing lists