Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes build when building against libXaw 1.0.16 since
xorg/lib/libxaw@d0fcbd97 changed the definition of XawListChange
in <X11/Xaw/List.h> from _Xconst char * to String *, where
String is defined as const if _CONST_X_STRING is defined.
This also clears up 105 of the 162 -Wdiscarded-qualifiers warnings.
Closes: #16
Fixes: bccb777 ("Fix -Wincompatible-pointer-types warning from gcc (issue #15)")
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Include standard headers for getpwnam() & crypt() instead of
local prototype redefinitions.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Matches what we already do in greeter/verify.c
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
It was only defined in the old Imakefiles, and not in any autoconf config
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
UNRELIABLE_SIGNALS was only defined if SYSV was defined
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
SIGNALS_RESET_WHEN_CAUGHT was only defined if SYSV was defined
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Solaris does not specify "const" on the first argument to PAM conversation
functions, nor the third argument to pam_get_item(). Linux PAM does both.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
chooser.c:280:26: warning: passing argument 2 of ‘XawListChange’ from
incompatible pointer type [-Wincompatible-pointer-types]
280 | XawListChange (list, newTable, size, 0, TRUE);
| ^~~~~~~~
| |
| char **
In file included from chooser.c:59:
/usr/include/X11/Xaw/List.h:170:27: note: expected ‘const char **’ but
argument is of type ‘char **’
170 | _Xconst char **list,
Closes: #15
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
AC_PROG_LIBTOOL was replaced by LT_INIT in libtool 2 in 2008,
so it's time to rely on it.
Clears autoconf warnings:
configure.ac:37: warning: The macro 'AC_LIBTOOL_DLOPEN' is obsolete.
configure.ac:37: You should run autoupdate.
m4/ltoptions.m4:113: AC_LIBTOOL_DLOPEN is expanded from...
configure.ac:37: the top level
configure.ac:37: warning: AC_LIBTOOL_DLOPEN: Remove this warning and the call to _LT_SET_OPTION when you
configure.ac:37: put the 'dlopen' option into LT_INIT's first parameter.
./lib/autoconf/general.m4:2434: AC_DIAGNOSE is expanded from...
m4/ltoptions.m4:113: AC_LIBTOOL_DLOPEN is expanded from...
configure.ac:37: the top level
configure.ac:38: warning: The macro 'AC_PROG_LIBTOOL' is obsolete.
configure.ac:38: You should run autoupdate.
m4/libtool.m4:100: AC_PROG_LIBTOOL is expanded from...
configure.ac:38: the top level
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Current template fails to rebuild the CI container with the current
Debian stable release (a new release since the last update)
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Oracle no longer includes this term in our copyright & license notices.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Avoid unnecessary library dependency when using a libc with these
functions included.
Also only links to the binary that needs them (xdm, not chooser or greeter)
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
This is the preferred usage form for libbsd, as it makes the code more
portable and requires no special includes for libbsd, by transparently
injects the needed standard headers that would be used on a BSD.
Signed-off-by: Guillem Jover <guillem@hadrons.org>
|
|
HAVE_ARC4RANDOM is defined under glibc 2.36 but this will attempt to include
<bsd/stdlib.h> from libbsd instead of <stdlib.h> from glibc
Signed-off-by: Callum Farmer <gmbr3@opensuse.org>
|
|
Resolves issue found by Oracle Parfait static analyzer:
Error: Uninitialised memory
Uninitialised memory [uninitialised-mem] (CWE 456):
Possible access to uninitialised memory referenced by 'new->fullname'
Memory 'new->fullname' accessed at byte offset '8'
at line 357 of app/xdm/chooser/chooser.c in function 'AddHostname'.
Write does not overlap at line 355
new allocated at line 329
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Most of these came from a mass bcopy() -> memmove() substitution
in 1993 with a commit comment of "ANSIfied for R6".
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Don't try to process returned data values that may not be set
Resolves Oracle Parfait issue:
Error: Uninitialised memory
Uninitialised memory variable [uninitialised-mem-var] (CWE 457):
Possible access to uninitialised memory referenced by variable 'length'
at line 238 of xdm/xdmcp.c in function 'all_query_respond'.
Path in callee avoiding write at line 237
length allocated at line 231
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Should never happen, but makes Oracle Parfait stop warning:
Error: Buffer overrun
Read Outside Array Bounds in STD C function [read-outside-array-bounds-call-stdc]:
Read outside array bounds in call to memmove. Buffer ((int8*)&in_addr) of size 256 is read at an offset of len
Array size is 256 bytes, index <= 1024
at line 130 of xdm/socket.c in function 'GetChooserAddr'.
called at line 197 of xdm/choose.c in function 'FormatChooserArgument' with lenp = &addr_len.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Bug: https://bugs.gentoo.org/384371
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Resolves warning from Oracle Parfait static analyzer:
Warning: Dangling pointer
Dangling pointer [dangling-pointer]:
Invalid pointer is accessible to caller via global addrs
at line 664 of xdm/auth.c in function 'doneAddrs'.
Invalid pointer accessible via global addrs at line 646
a accessed from addrs at line 658 in function 'doneAddrs'
a freed with free at line 662
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Oracle Parfait static analyser points out it wasn't clear if cast or
arithmetic operation in arguments at lines 410, 416, 431, & 433 was
intended to be done first, so we'll make it explicit.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
libsystemd-daemon was merged into libsystemd in 2014, for systemd-209.
Fixes: https://gitlab.freedesktop.org/xorg/app/xdm/-/issues/9
Signed-off-by: Matt Turner <mattst88@gmail.com>
|
|
Found by using:
codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93359
while avoiding https://bugs.freedesktop.org/show_bug.cgi?id=107368
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
changing the printf-like format as strlen returns an unsigned type
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|