summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2024-02-17unifdef hpuxHEADmasterAlan Coopersmith1-35/+0
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2024-02-01Autest.c includes Xauth.h that depends on other headers.Thierry LARONDE1-0/+1
Xauth.h includes headers provided by xproto. So add for test also the dependency.
2024-02-01Allow to use POSIX.2 compliant sed(1)Thierry LARONDE1-2/+3
POSIX.2 sed(1) doesn't allow file to be anywhere else than, if given, last. Allow to use whatever compliant sed(1) is there, not only GNU sed(1).
2024-02-01Also test for explicit_memset(3) supportThierry LARONDE4-3/+9
explicit_bzero(3) is not provided by every system, and some provide explicit_memset(3) that is in the process of being standardized (furthermore, POSIX deprecates bzero(3) since memset(3) can do same or more).
2024-02-01Gcc < 11 doesn't allow opt. deallocator arg for __attribute((malloc))Thierry LARONDE1-2/+2
2023-09-16gitlab CI: run meson build with both gcc & clang compilersAlan Coopersmith1-5/+14
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2023-09-12Allow building with clang after aec9d7266777e0b9243ef0f112fe0e07256bd446Alex Richardson1-1/+6
Clang also supports the malloc attribute but unlike GCC does not accept the deallocator argument. This has been reported as https://github.com/llvm/llvm-project/issues/51607 Signed-off-by: Alex Richardson <arichardson@FreeBSD.org>
2023-07-30Xauth.h: add access & malloc attributes to function prototypesAlan Coopersmith1-4/+31
May aid in static analysis by gcc Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2023-07-13add Meson build systemDylan Baker6-6/+199
This is, I think, equal to the autotools build-system in every practical way. The man pages have hardcoded numbers. I think this is okay, as all modern operating systems seem to use the same man page numbering now. I've also chosen to not generate a config.h file with Meson. This makes using libXau as a subproject safer, since we don't have to worry about conflicting config.h files, plus there are only a couple of declarations that are actually used. This also saves some configure time in writing out an additional file.
2023-03-25Set close-on-exec when opening filesAlan Coopersmith3-3/+20
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2023-02-25Remove "All rights reserved" from Oracle copyright noticesAlan Coopersmith1-1/+1
Oracle no longer includes this term in our copyright & license notices. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2023-02-06Require LT_INIT from libtool 2 instead of deprecated AC_PROG_LIBTOOLAlan Coopersmith2-2/+4
AC_PROG_LIBTOOL was replaced by LT_INIT in libtool 2 in 2008, so it's time to rely on it. configure.ac:35: warning: The macro `AC_LIBTOOL_WIN32_DLL' is obsolete. configure.ac:35: You should run autoupdate. aclocal.m4:11889: AC_LIBTOOL_WIN32_DLL is expanded from... configure.ac:35: the top level configure.ac:35: warning: AC_LIBTOOL_WIN32_DLL: Remove this warning and the call to _LT_SET_OPTION when you configure.ac:35: put the 'win32-dll' option into LT_INIT's first parameter. ./lib/autoconf/general.m4:2434: AC_DIAGNOSE is expanded from... aclocal.m4:11889: AC_LIBTOOL_WIN32_DLL is expanded from... configure.ac:35: the top level configure.ac:36: warning: The macro `AC_PROG_LIBTOOL' is obsolete. configure.ac:36: You should run autoupdate. aclocal.m4:3465: AC_PROG_LIBTOOL is expanded from... configure.ac:36: the top level Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-12-08libXau 1.0.11libXau-1.0.11Matt Turner1-1/+1
Signed-off-by: Matt Turner <mattst88@gmail.com>
2022-11-29configure: Use AC_SYS_LARGEFILE to enable large file supportMatt Turner1-0/+3
Signed-off-by: Matt Turner <mattst88@gmail.com>
2022-08-26libXau 1.0.10libXau-1.0.10Alan Coopersmith1-2/+2
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-17XauReadAuth: move failure handling code to a common code blockAlan Coopersmith1-27/+28
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-09Remove unnnecessary casts from malloc() and free() callsAlan Coopersmith2-2/+2
These are not needed in C89 and later. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-09Autest.c: Fix -Wdiscarded-qualifiers warningsAlan Coopersmith1-4/+7
Autest.c: In function ‘main’: Autest.c:38:21: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] 38 | char *name = "XAU-TEST-1"; | ^~~~~~~~~~~~ Autest.c:39:21: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] 39 | char *data = "Do not begin the test until instructed to do so."; | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Autest.c:57:23: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] 57 | test_data.address = ""; | ^ Autest.c:59:22: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers] 59 | test_data.number = ""; | ^ Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-09Fix spelling/wording issuesAlan Coopersmith4-4/+4
Found by using: codespell --builtin clear,rare,usage,informal,names Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-09Build xz tarballs instead of bzip2Alan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-07-09Use explicit_bzero if availableTobias Stoeckmann3-1/+13
Optimizing compilers may remove the bzero call because it is followed by free. The function explicit_bzero avoids this optimization. Use it if it is available. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
2022-07-09gitlab CI: add a basic build testAlan Coopersmith1-0/+98
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2019-02-10libXau 1.0.9libXau-1.0.9Alan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2018-12-07Update configure.ac bug URL for gitlab migrationAlan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2017-10-30AuRead.c: remove redundant null check on calling free()walter harms1-9/+9
this removes simply unneeded code from XauReadAuth Signed-off-by: Walter Harms <wharms@bfs.de> Reviewed-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
2017-10-30Au FileName.c: remove redundant null check on calling free()walter harms1-2/+1
remove redundant null check on calling free() Signed-off-by: Walter Harms <wharms@bfs.de> Reviewed-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
2017-10-30AuDispose.c:remove redundant null check on calling free()walter harms1-3/+3
redundant null check on auth->address calling free() redundant null check on auth->number calling free() redundant null check on auth->name calling free() Signed-off-by: Walter Harms <wharms@bfs.de> Reviewed-by: Daniel Martin <consume.noise@gmail.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
2017-10-20Avoid out of boundary read accessTobias Stoeckmann1-1/+1
If the environment variable HOME is empty, XauFileName triggers an out of boundary read access (name[1]). If HOME consists of a single character relative path, the output becomes unexpected, because "HOME=a" leads to "a.Xauthority" instead of "a/.Xauthority". Granted, a relative HOME path leads to trouble in general, the code should properly return "a/.Xauthority" nonetheless. Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2017-01-26autogen: add default patch prefixMihail Konev1-0/+3
Signed-off-by: Mihail Konev <k.mvc@ya.ru>
2017-01-26autogen.sh: use quoted string variablesEmil Velikov1-4/+4
Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent fall-outs, when they contain space. Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net> Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
2017-01-26autogen.sh: use exec instead of waiting for configure to finishPeter Hutterer1-1/+1
Syncs the invocation of configure with the one from the server. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
2014-01-02Silence a benign static analysis warning with an assert of allocation sizeJeremy Huddleston Sequoia1-0/+2
AuFileName.c:72:8: warning: Call to 'malloc' has an allocation size of 0 bytes buf = malloc (size); ^~~~~~~~~~~~~ Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
2013-09-29XauFileName: always go through buf allocation if buf is NULLAlan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-09-29XauFileName: reset bsize when malloc failedFuminobu TAKEYAMA1-1/+3
https://bugs.freedesktop.org/show_bug.cgi?id=69929 Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-05-24libXau 1.0.8libXau-1.0.8Alan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-03-30Use remove() instead of unlink() to delete lock filesAlan Coopersmith2-4/+4
In normal cases, no difference. If someone has accidentally created a directory with those names, this can save you from needing to fsck to clean it up on some filesystems (like UFS). Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-03-30Call memcmp() instead of defining our own equivalentAlan Coopersmith2-22/+8
Stops casting len parameters to int in callers so that this change doesn't introduce warnings that we're taking unsigned values, casting them to signed, then passing to a function that wants them unsigned. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-03-30Replace strcpy+strcat pairs with snprintf callsAlan Coopersmith3-12/+7
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-03-30Check for configure's HAVE_UNISTD_H instead of X_NOT_POSIXAlan Coopersmith1-8/+5
Drops fallback prototype of sleep() for non-POSIX, non-Windows systems Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-03-30Use configure check for pathconf instead of X_NOT_POSIX #defineAlan Coopersmith2-4/+7
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-01-15autogen.sh: Implement GNOME Build APIColin Walters1-1/+3
http://people.gnome.org/~walters/docs/build-api.txt Signed-off-by: Adam Jackson <ajax@redhat.com>
2013-01-15configure: Remove AM_MAINTAINER_MODEAdam Jackson1-1/+0
Signed-off-by: Adam Jackson <ajax@redhat.com>
2013-01-12Convert INCLUDES to AM_CPPFLAGS in preparation for automake-1.14 obsoletionAlan Coopersmith1-2/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-01-04unifdef -U__UNIXOS2__Alan Coopersmith1-3/+0
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2012-12-21Tell clang to shut up about the padding in struct xauthAlan Coopersmith1-0/+12
We can't clean it up without breaking ABI, so disable the warnings for it: ./include/X11/Xauth.h:33:19: warning: padding struct 'struct xauth' with 4 bytes to align 'address' [-Wpadded] char *address; ^ ./include/X11/Xauth.h:35:19: warning: padding struct 'struct xauth' with 6 bytes to align 'number' [-Wpadded] char *number; ^ ./include/X11/Xauth.h:37:19: warning: padding struct 'struct xauth' with 6 bytes to align 'name' [-Wpadded] char *name; ^ ./include/X11/Xauth.h:39:18: warning: padding struct 'struct xauth' with 6 bytes to align 'data' [-Wpadded] char *data; ^ Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2012-12-21Clean up some clang warnings about sign conversionAlan Coopersmith3-7/+7
fread & fwrite are defined as taking size_t arguments (an unsigned type), so stop casting their arguments to a signed int just to confuse things. Fixes warnings: AuFileName.c:69:59: warning: implicit conversion loses integer precision: 'unsigned long' to 'int' [-Wshorten-64-to-32] size = strlen (name) + strlen(&slashDotXauthority[1]) + 2; ~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~ AuRead.c:58:44: warning: implicit conversion changes signedness: 'int' to 'size_t' (aka 'unsigned long') [-Wsign-conversion] if (fread (data, (int) sizeof (char), (int) len, file) != len) { ~~~~~ ^~~~~~~~~ AuWrite.c:49:46: warning: implicit conversion changes signedness: 'int' to 'size_t' (aka 'unsigned long') [-Wsign-conversion] if (fwrite (string, (int) sizeof (char), (int) count, file) != count) ~~~~~~ ^~~~~~~~~~~ Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
2012-08-23Remove a use of tab stops.Eric S. Raymond1-26/+24
2012-04-20Free XauFileName() static buffer at exitChase Douglas1-1/+16
XauFileName() may allocate and return a static buffer. The only way to ensure it is freed is to deallocate it when the program exits. Signed-off-by: Chase Douglas <chase.douglas@canonical.com> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2012-03-06libXau 1.0.7libXau-1.0.7Alan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2011-09-20Improve the handling of "normal" locking collisions.Matthew D. Fuller1-1/+1
EEXIST is a perfectly normal and expected errno for open(O_CREAT | O_EXCL), and is a signal to loop around and retry, not return with an error. Signed-off-by: Matthew D. Fuller <fullermd@over-yonder.net> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-14 03:30:12 +0000