summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* libICE 1.0.9libICE-1.0.9Alan Coopersmith2014-06-061-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Delete unused name variable in register.cAlan Coopersmith2013-12-241-6/+2
| | | | | | | | | | Found by cppcheck 1.62: [src/register.c:84]: (style) Variable 'name' is assigned a value that is never used. [src/register.c:182]: (style) Variable 'name' is assigned a value that is never used. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Free iceConn->connection_string when unwinding after malloc failsAlan Coopersmith2013-12-241-0/+2
| | | | | | | Found by cppcheck 1.62: [src/accept.c:113]: (error) Memory leak: iceConn.connection_string Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Use arc4random when available to produce the auth cookie.Matthieu Herrb2013-09-122-2/+12
| | | | | | | | arc4random() and associated functions can be found in libbsd on GNU/Linux systems. Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Make STORE_STRING cast strlen result to CARD16 when storing in CARD16Alan Coopersmith2013-08-081-1/+1
| | | | | | | | | | | | | | | Clears a number of clang warnings of the form: connect.c:328:6: warning: implicit conversion loses integer precision: 'size_t' (aka 'unsigned long') to 'CARD16' (aka 'unsigned short') [-Wconversion] STORE_STRING (pData, _IceAuthNames[i]); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ./ICElibint.h:173:19: note: expanded from macro 'STORE_STRING' CARD16 _len = strlen (_string); \ ~~~~ ^~~~~~~~~~~~~~~~ Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Cast assignments in IceErrorHeader() macroAlan Coopersmith2013-08-081-4/+4
| | | | | | | Clears many clang warnings about implicit conversions losing integer precision, such as from storing ints into CARD8 or CARD16. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Make write_string call write_counted_string instead of copying itAlan Coopersmith2013-08-081-7/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Mark input arguments to write_string functions as constAlan Coopersmith2013-08-081-4/+4
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Stop casting return values from mallocAlan Coopersmith2013-08-0817-69/+46
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Get rid of casts to (char *) in calls to free()Alan Coopersmith2013-08-0812-46/+46
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Convert remaining sprintf() call to snprintf()Alan Coopersmith2013-07-201-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Convert strcpy/strcat pairs to snprintf callsAlan Coopersmith2013-07-201-13/+6
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Fix some clang warnings about integer sign/size conversionsAlan Coopersmith2013-07-203-16/+20
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* unifdef WORD64Alan Coopersmith2013-07-203-156/+0
| | | | | | | Remove leftover remnants of CRAY support, which still had some functions consisting solely of /* NOT IMPLEMENTED YET */ comments. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Constify filename argument to IceLockAuthFile & IceUnlockAuthFileAlan Coopersmith2013-07-202-4/+4
| | | | | | | | Needed to fix const string warnings in iceauth - functions already copy provided arguments to temporary local buffer for modifications. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
* 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>
* unifdef -U__UNIXOS2__Alan Coopersmith2013-01-044-22/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* libICE 1.0.8libICE-1.0.8Alan Coopersmith2012-03-021-1/+1
| | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* informaltable cleanupMatt Dew2012-01-092-320/+323
| | | | | | | | | | | | | | | | On certain tables, add top and bottom borders to table header and a bottom border to the table. This matches what those tables in the old pdfs looked like. the <?dbfo keep-together='always'> prevents tables from splitting across pages. Useful for tiny tables. Converting the colwidth to a floating point, IE, 1* -> 1.0* cleans up these build errors: WARNING: table-layout="fixed" and column-width unspecified => falling back to proportional-column-width(1) Signed-off-by: Matt Dew <marcoz@osource.org>
* Plug minor memory leak in unusual path through ProcessConnectionSetupAlan Coopersmith2011-11-301-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error: Memory leak (CWE 401) Memory leak of pointer 'release' allocated with malloc((_len + 1)) at line 1100 of src/process.c in function 'ProcessConnectionSetup'. 'release' allocated at line 920 with malloc((_len + 1)). release leaks when _i >= hisAuthCount at line 925 and i >= _IceAuthCount at line 973 and found != 0 at line 998 and status != 0 at line 1053 and status != 1 at line 1070 and accept_setup_now == 0 at line 1082 and i >= hisAuthCount at line 1093. Memory leak of pointer 'vendor' allocated with malloc((_len + 1)) at line 1100 of src/process.c in function 'ProcessConnectionSetup'. 'vendor' allocated at line 919 with malloc((_len + 1)). vendor leaks when _i >= hisAuthCount at line 925 and i >= _IceAuthCount at line 973 and found != 0 at line 998 and status != 0 at line 1053 and status != 1 at line 1070 and accept_setup_now == 0 at line 1082 and i >= hisAuthCount at line 1093. [ This bug was found by the Parfait 0.3.7 bug checking tool. For more information see http://labs.oracle.com/projects/parfait/ ] Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Constify protocol, vendor & release string args to IceRegisterForProtocol*Alan Coopersmith2011-11-223-20/+30
| | | | | | | | | | | | | Needed to resolve gcc -Wwrite-strings warnings in callers. These functions only pass the strings to strcmp before calling strdup to make their own private copy for storing away. While fixing the API docs to match, also fix them to match the existing function prototypes, where there were several errors before (including just plain missing most of the args to IceRegisterForProtocolReply). Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Remove ancient workaround for System V/386 Release 4.2 compiler bugAlan Coopersmith2011-11-111-11/+1
| | | | | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Matthieu Herrb <matthieu.herrb@laas.fr> Reviewed-by: walter <wharms@bfs.de> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Fix gcc -Wwrite-strings warnings in process.cAlan Coopersmith2011-11-111-11/+13
| | | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Fix gcc -Wwrite-strings warnings in AuthNames handlingAlan Coopersmith2011-11-115-40/+41
| | | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* constify arguments to IceGetAuthFileEntryAlan Coopersmith2011-11-113-9/+9
| | | | | | | Needed to clear gcc -Wwrite-strings warnings in callers Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Fix gcc -Wwrite-strings warnings in _IceError* functionsAlan Coopersmith2011-11-112-10/+10
| | | | | Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Fix gcc -Wwrite-strings warnings in _IceDefaultErrorHandlerAlan Coopersmith2011-11-111-17/+18
| | | | | | | | | | Had to split char *str into two variables, const char *str for the string literals just being passed to fprintf etal. and char *estr for use by EXTRACT_STRING for the results of malloc calls that get written to and then freed. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Cleanup IDs and links in docMatt Dew2011-10-042-321/+308
| | | | | | | | | | 1 - fix the capitalization of the ID attributes to match either the <title> or <funcdef> string it goes with. 2 - fix any <linkend>'s that were affected by 1. 3 - any <function> in the docs that has an actual funcdef, will become an olink. Signed-off-by: Matt Dew <marcoz@osource.org>
* As of 1997, X Window System is a trademark of The Open Group.Gaetan Nadon2011-09-191-2/+2
| | | | Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Strip trailing whitespaceAlan Coopersmith2011-09-1617-57/+57
| | | | | | | 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>
* specs: remove <productnumber> which is not used by defaultGaetan Nadon2011-09-151-1/+0
| | | | | | | | | This element is not rendered by default on the title. A template customization is required to display it. X Window System does not have a product number. Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* specs: use DocBook suggested markup for CopyrightsGaetan Nadon2011-09-152-5/+8
| | | | | | | | | | Puts the statement on a single line, using commas to separate years --> Copyright © 1993, 1994, 1996 X Consortium As opposed to 3 double-spaced lines. Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* specs: remove orphan <corpname>Gaetan Nadon2011-09-152-2/+0
| | | | | | | It does not display on the title page and is redundant. No visible change on the doc. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* specs: use the &fullrelvers; entity to set X11 release informationGaetan Nadon2011-09-152-5/+10
| | | | | Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* ice.xml: version number is 1.1, not 1.0Gaetan Nadon2011-09-151-1/+1
| | | | | | | This is what it was before the conversion from roff to docbook. Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* ice.xml: fix orphan affiliationGaetan Nadon2011-09-111-1/+1
| | | | Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* ICElib doc: remove empty revision list and specify the doc version numberGaetan Nadon2011-09-111-1/+1
| | | | | | | Revision histories are not used, only 3 docs out of 63 have one. Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* docbook.am: embed css styles inside the HTML HEAD elementGaetan Nadon2011-09-081-2/+0
| | | | | | | | | | | | Rather than referring to the external xorg.css stylesheet, embed the content of the file in the html output produced. This is accomplished by using version 1.10 of xorg-xhtml.xsl. This makes the whole html docs tree much more relocatable. In addition, it eliminates xorg.css as a runtime file which makes xorg-sgml-doctools a build time only package. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* docbook.am: global maintenance update - entities, images and olinkingGaetan Nadon2011-09-071-52/+62
| | | | | | | | | | | | | Adding support in libX11 for html chunking caused a reorg of docbook.am as well as the xorg-sgml-doctools masterdb for olinking. The parameter img.src.path is added for pdf images. A searchpath to the root builddir is added for local entities, if present. The docbook.am makefile hides all the details and is identical for all 22 modules having DocBook documentation. It is included by a thin Makefile.am which requires no docbook knowledge. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Install xml versions of specs even if HAVE_XMLTO is falseGaetan Nadon2011-06-123-4/+2
| | | | | | | DocBook/XML input source is also a usefull output format that can be viewed with an XML viewer or editor and by some O/S help system. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Install target dbs alongside generated documentsGaetan Nadon2011-06-112-3/+3
| | | | | | | | | | | | This matches a change in xorg-sgml-docs whereby the masterdb will look for the target dbs into the same location as the generated documents. The target dbs are now installed alongside the generated documents. Previously they are installed in $prefix/sgml/X11/dbs alongside masterdb which has the potential of installing outside the package prefix and cause distcheck to fail when user does not have write permission in this package. Requires XORG_CHECK_SGML_DOCTOOLS(1.8) which was released 2011-06-11
* add id attributes to funcsynopsis to allow other docs to olink to themMatt Dew2011-06-041-77/+77
| | | | | | Signed-off-by: Matt Dew <marcoz@osource.org> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Gaetan Nadon <memsize@videotron.ca>
* IceWritePad: Zero fill pad bytes used when buffer is fullAlan Coopersmith2011-04-211-3/+3
| | | | | | | | | | Should be rarely hit, since it's only in the case where most of the message fits in the buffer, but there's not room left for the padding bytes, but better than sending uninitialized bytes off the stack when it happens. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* Fix reads outside array bounds in error handlersAlan Coopersmith2011-04-211-4/+4
| | | | | | | | | | | | | | | | | Error: Buffer overrun Read outside array bounds (CWE 125): In call to memcpy(<unknown>, &mOp, 8), (size(&mOp) < (unsigned) 8) Array size is 1 bytes at line 296 of src/error.c in function '_IceErrorMajorOpcodeDuplicate'. Error: Buffer overrun Read outside array bounds (CWE 125): In call to memcpy(<unknown>, &maj, 8), (size(&maj) < (unsigned) 8) Array size is 1 bytes at line 346 of src/error.c in function '_IceErrorBadMajor'. [ This bug was found by the Parfait 0.3.7 bug checking tool. For more information see http://labs.oracle.com/projects/parfait/ ] Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
* doc: xorg-sgml-doctools package at version 1.7 is requiredGaetan Nadon2011-03-301-1/+1
| | | | | | This version contains support for external references Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Documentation: add Docbook external references supportGaetan Nadon2011-03-236-108/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | When writing technical documentation, it is often necessary to cross reference to other information. When that other information is not in the current document, additional support is needed, namely <olink>. A new feature with version 1.7 of xorg-sgml-doctools adds references to other documents within or outside this package. This patch adds technical support for this feature but does not change the content of the documentation as seen by the end user. Each book or article must generate a database containing the href of sections that can be referred to from another document. This database is installed in DATAROOTDIR/sgml/X11/dbs. There is a requirement that the value of DATAROOTDIR for xorg-sgml-doctools and for the package documentation is the same. This forms a virtual document tree. This database is consulted by other documents while they are being generated in order to fulfill the missing information for linking. Refer to the xorg-sgml-doctools for further technical information. Co-authored-by: Matt Dew <marcoz@osource.org> Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* Docbook: change the book id to match the xml file basenameGaetan Nadon2011-02-251-1/+1
| | | | | | This is required for the up-coming external references support. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: splitting ICE and XTRANS compiler options is not requiredGaetan Nadon2011-02-022-4/+2
| | | | | | Simplify configuration by using a single PKG_CHECK_MODULES statement. Signed-off-by: Gaetan Nadon <memsize@videotron.ca>
* config: comment, minor upgrade, quote and layout configure.acGaetan Nadon2011-02-021-14/+12
| | | | | | | | | | | | | 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. 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>
View Lorry Controller Status