summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Set close-on-exec when opening filesHEADmasterAlan Coopersmith2023-03-253-3/+20
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Remove "All rights reserved" from Oracle copyright noticesAlan Coopersmith2023-02-251-1/+1
| | | | | | Oracle no longer includes this term in our copyright & license notices. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Require LT_INIT from libtool 2 instead of deprecated AC_PROG_LIBTOOLAlan Coopersmith2023-02-062-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>
* libXau 1.0.11libXau-1.0.11Matt Turner2022-12-081-1/+1
| | | | Signed-off-by: Matt Turner <mattst88@gmail.com>
* configure: Use AC_SYS_LARGEFILE to enable large file supportMatt Turner2022-11-291-0/+3
| | | | Signed-off-by: Matt Turner <mattst88@gmail.com>
* libXau 1.0.10libXau-1.0.10Alan Coopersmith2022-08-261-2/+2
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* XauReadAuth: move failure handling code to a common code blockAlan Coopersmith2022-07-171-27/+28
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Remove unnnecessary casts from malloc() and free() callsAlan Coopersmith2022-07-092-2/+2
| | | | | | These are not needed in C89 and later. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Autest.c: Fix -Wdiscarded-qualifiers warningsAlan Coopersmith2022-07-091-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>
* Fix spelling/wording issuesAlan Coopersmith2022-07-094-4/+4
| | | | | | | Found by using: codespell --builtin clear,rare,usage,informal,names Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Build xz tarballs instead of bzip2Alan Coopersmith2022-07-091-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Use explicit_bzero if availableTobias Stoeckmann2022-07-093-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>
* gitlab CI: add a basic build testAlan Coopersmith2022-07-091-0/+98
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* libXau 1.0.9libXau-1.0.9Alan Coopersmith2019-02-101-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Update configure.ac bug URL for gitlab migrationAlan Coopersmith2018-12-071-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* AuRead.c: remove redundant null check on calling free()walter harms2017-10-301-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>
* Au FileName.c: remove redundant null check on calling free()walter harms2017-10-301-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>
* AuDispose.c:remove redundant null check on calling free()walter harms2017-10-301-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>
* Avoid out of boundary read accessTobias Stoeckmann2017-10-201-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>
* autogen: add default patch prefixMihail Konev2017-01-261-0/+3
| | | | Signed-off-by: Mihail Konev <k.mvc@ya.ru>
* autogen.sh: use quoted string variablesEmil Velikov2017-01-261-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>
* autogen.sh: use exec instead of waiting for configure to finishPeter Hutterer2017-01-261-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>
* Silence a benign static analysis warning with an assert of allocation sizeJeremy Huddleston Sequoia2014-01-021-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>
* XauFileName: always go through buf allocation if buf is NULLAlan Coopersmith2013-09-291-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* XauFileName: reset bsize when malloc failedFuminobu TAKEYAMA2013-09-291-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>
* libXau 1.0.8libXau-1.0.8Alan Coopersmith2013-05-241-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Use remove() instead of unlink() to delete lock filesAlan Coopersmith2013-03-302-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>
* Call memcmp() instead of defining our own equivalentAlan Coopersmith2013-03-302-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>
* Replace strcpy+strcat pairs with snprintf callsAlan Coopersmith2013-03-303-12/+7
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Check for configure's HAVE_UNISTD_H instead of X_NOT_POSIXAlan Coopersmith2013-03-301-8/+5
| | | | | | Drops fallback prototype of sleep() for non-POSIX, non-Windows systems Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Use configure check for pathconf instead of X_NOT_POSIX #defineAlan Coopersmith2013-03-302-4/+7
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* autogen.sh: Implement GNOME Build APIbaserock/morphColin Walters2013-01-151-1/+3
| | | | | | http://people.gnome.org/~walters/docs/build-api.txt Signed-off-by: Adam Jackson <ajax@redhat.com>
* configure: Remove AM_MAINTAINER_MODEAdam Jackson2013-01-151-1/+0
| | | | Signed-off-by: Adam Jackson <ajax@redhat.com>
* Convert INCLUDES to AM_CPPFLAGS in preparation for automake-1.14 obsoletionAlan Coopersmith2013-01-121-2/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* unifdef -U__UNIXOS2__Alan Coopersmith2013-01-041-3/+0
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Tell clang to shut up about the padding in struct xauthAlan Coopersmith2012-12-211-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>
* Clean up some clang warnings about sign conversionAlan Coopersmith2012-12-213-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>
* Remove a use of tab stops.Eric S. Raymond2012-08-231-26/+24
|
* Free XauFileName() static buffer at exitChase Douglas2012-04-201-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>
* libXau 1.0.7libXau-1.0.7Alan Coopersmith2012-03-061-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Improve the handling of "normal" locking collisions.Matthew D. Fuller2011-09-201-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>
* Strip trailing whitespaceAlan Coopersmith2011-09-163-5/+5
| | | | | | | Performed with: find * -type f | xargs perl -i -p -e 's{[ \t]+$}{}' git diff -w & git diff -b show no diffs from this change Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* config: comment, minor upgrade, quote and layout configure.acGaetan Nadon2011-02-021-32/+39
| | | | | | | | | | | | | | | | Group statements per section as per Autoconf standard layout Quote statements where appropriate. Autoconf recommends not using dnl instead of # for comments Use AC_CONFIG_FILES to replace the deprecated AC_OUTPUT with parameters. Use AC_PROG_LIBTOOL to replace the deprecated AM_PROG_LIBTOOL Add AC_CONFIG_SRCDIR([Makefile.am]) where missing No functional configuration changes This helps automated maintenance and release activities. Details can be found in http://wiki.x.org/wiki/NewModuleGuidelines Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: replace deprecated AM_CONFIG_HEADER with AC_CONFIG_HEADERSGaetan Nadon2011-01-281-1/+1
| | | | Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: replace deprecated AC_HELP_STRING with AS_HELP_STRINGGaetan Nadon2011-01-281-1/+1
| | | | | | This silences an Automake warning. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: remove already included statementsGaetan Nadon2011-01-281-3/+0
| | | | | | | AC_PROG_INSTALL and AC_PROG_SED are included in XORG_DEFAULT_OPTIONS. AC_PROG_MAKE_SET is included by AM_INIT_AUTOMAKE Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: remove AC_PROG_CC as it overrides AC_PROG_C_C99Gaetan Nadon2011-01-271-1/+0
| | | | | | | | XORG_STRICT_OPTION from XORG_DEFAULT_OPTIONS calls AC_PROG_C_C99. This sets gcc with -std=gnu99. If AC_PROG_CC macro is called afterwards, it resets CC to gcc. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Purge cvs tags.Jesse Adkins2010-10-0611-35/+0
| | | | | Signed-off-by: Jesse Adkins <jesserayadkins@gmail.com> Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Xau.man: replace hard-coded 3 with __libmansuffix__Gaetan Nadon2010-08-111-1/+1
| | | | Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Xau.man: remove whitespace as reported by git diff checkGaetan Nadon2010-08-111-23/+23
| | | | Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
View Lorry Controller Status