Fix nisplus-alias compile with gcc7
Commit Message
I think the patch is fairly obvious: return error if there is no
name as opposed to if there isn't.
We follow up this check with various strlen, printf etc. It's
the latter for which gcc7 mainline warns that we pass NULL to %s,
and with -Werror that gets noticed.
r~
* nis/nss_nisplus/nisplus-alias.c (_nss_nisplus_getaliasbyname_r):
Fix test for NULL name.
---
nis/nss_nisplus/nisplus-alias.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Wed, 21 Dec 2016, Richard Henderson wrote:
> I think the patch is fairly obvious: return error if there is no
> name as opposed to if there isn't.
I've taken this statement of obviousness as consensus to commit my
(identical) patch
<https://sourceware.org/ml/libc-alpha/2016-12/msg00771.html>. Bug 20987
remains open for the various apparently unbounded stack allocations for
NIS+.
@@ -291,7 +291,7 @@ _nss_nisplus_getaliasbyname_r (const char *name, struct aliasent *alias,
return status;
}
- if (name != NULL)
+ if (name == NULL)
{
*errnop = EINVAL;
return NSS_STATUS_UNAVAIL;