{"ref":"refs/heads/master","commits":[{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"feebf6746374aa04b12e9e3e51313a3a82c03530","message":"Limit the number of screens Xvfb will attempt to allocate memory for\n\nCommit f9e3a2955d2ca7 removing the MAXSCREEN limit left the screen\nnumber too unlimited, and allowed any positive int for a screen number:\n\nXvfb :1 -screen 2147483647 1024x1024x8\n\nFatal server error:\nNot enough memory for screen 2147483647\n\nFound by Parfait 0.3.7:\nError: Integer overflow (CWE 190)\n   Integer parameter of memory allocation function realloc() may overflow due to multiplication with constant value 1112\n        at line 293 of hw/vfb/InitOutput.c in function 'ddxProcessArgument'.\n\nSince the X11 connection setup only has a CARD8 for number of SCREENS,\nlimit to 255 screens, which is also low enough to avoid overflow on the\nsizeof(*vfbScreens) * (screenNum + 1) calculation for realloc.\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Jamey Sharp <jamey@minilop.net>","url":"https://api.github.com/repos/shadchin/xserver/commits/feebf6746374aa04b12e9e3e51313a3a82c03530"},{"author":{"email":"8a8cc541338200f67d4d0bcb65c78b6166649462@videotron.ca","name":"Gaetan Nadon"},"sha":"fd976e4051e15ab2d01ec0bf89ff26926d4ed04e","message":"config: conftest needs an include directive to locate X11/Xfuncproto.h\n\nThe compiler option -fvisibility=hidden is erroneously missing\ndue to a faulty configuration test. The gcc command is unable to\nlocate X11/Xfuncproto.h unless the build occurs on a\nsystem where X11 development headers are installed.\n\nconfigure:21294: checking for symbol visibility support\nconfigure:21323: gcc -std=gnu99 -c -g -O2 -fvisibility=hidden  conftest.c >&5\nconftest.c:144:28: fatal error: X11/Xfuncproto.h: No such file or directory\n\nThe solution is to add an include directive to obtain the location\nof X11/Xfuncproto.h which may or may not be the system installed headers.\n\nReviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nSigned-off-by: Gaetan Nadon <memsize@videotron.ca>\nSigned-off-by: Keith Packard <keithp@keithp.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/fd976e4051e15ab2d01ec0bf89ff26926d4ed04e"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"84207def93e4489df8dca8e044ce544be8a3f845","message":"Disable building of tests requiring DDX functions when Xorg is not built\n\nSome test cases require linking with some sort of DDX - ideally we'd\nhave a stub ddx for testing, but for now, since we link with the Xorg\nddx, disable those tests when configured with --disable-xorg\n\nFixes https://bugs.freedesktop.org/show_bug.cgi?id=43320\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Dan Nicholson <dbn.lists@gmail.com>\nReviewed-by: Jeremy Huddleston <jeremyhu@apple.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/84207def93e4489df8dca8e044ce544be8a3f845"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"4be68b03128e958d2a6dc1b7feb3587329b9561b","message":"Remove unused function checks from configure.ac & include/*.h.in\n\nThe code that used getisax to check for MMX support was moved to pixman\nand removed from the X server by commit eb2d7fe02f9cbc.\n\nThe code that used HAVE_MKSTEMP was deleted by the Xprint removal in\ncommit 1c8bd318fbaf.\n\nAll alloca calls were removed by the patch series end in commit 5e363500c8,\nand used custom X checks instead of the autoconf HAVE_ALLOCA anyway.\n\nI can find no record of HAVE_GETUID, HAVE_GETEUID, HAVE_LINK, HAVE_MEMMOVE,\nHAVE_MEMSET, HAVE_STRCHR, HAVE_STRRCHR, HAVE_GETOPT, HAVE_GETOPT_LONG,\nHAVE_DOPRNT, or HAVE_VPRINTF ever being used, and the calls to those\nfunctions are not wrapped in #ifdefs.\n(Most of those are in our baseline requirements of C89 & Unix98 anyway.)\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Adam Jackson <ajax@redhat.com>\nReviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>\nReviewed-by: Gaetan Nadon <memsize@videotron.ca>","url":"https://api.github.com/repos/shadchin/xserver/commits/4be68b03128e958d2a6dc1b7feb3587329b9561b"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"d829a7c5cb42c979b58f3547136df5b05d906423","message":"Move to autoconf standard function name checks & defines\n\nReplace multiple methods of checking for functions with AC_CHECK_FUNCS\nReplace multiple methods of selecting fallback funcs with AC_REPLACE_FUNCS\nReplace HAS_* and NEED_* #defines with autogenerated HAVE_*\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Adam Jackson <ajax@redhat.com>\nReviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>\nReviewed-by: Gaetan Nadon <memsize@videotron.ca>","url":"https://api.github.com/repos/shadchin/xserver/commits/d829a7c5cb42c979b58f3547136df5b05d906423"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"8b6a75009746db4b8e2e180210ac097e35136bcd","message":"Fix gcc warnings about redundant declarations of fallback functions\n\nEnsure ffs, strndup, strlcat, etc. aren't defined by our headers\nif they're already defined in the system headers.\n\nThis does export the HAVE_FFS, HAVE_STRNDUP, etc. definitions to drivers,\nbut if you built the Xserver with a libc that had those, and then build\nthe drivers with a less capable libc, you're going to have problems anyway,\nand this should solve some reported problems with conflicts between our\nstrndup definition and gcc magic for it.\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Adam Jackson <ajax@redhat.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/8b6a75009746db4b8e2e180210ac097e35136bcd"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"e4dcf580f007d6adcf5b0c639865d7aaab1a319d","message":"LoaderOpen returns either a valid pointer or NULL, so don't check for < 0\n\nFixes Sun cc warning that was recently elevated to error by the\nstricter default CFLAGS changes to xorg-macros:\n\n\"loadmod.c\", line 914: improper pointer/integer combination: op \"<\"\n\nShould have been changed when commit ab7f057ce9df4e905b12 changed the\nLoaderOpen return type from int to void *.\n\nChanges log message when file is found but dlopen() fails from:\n (EE) LoadModule: Module dbe does not have a dbeModuleData data object.\n (EE) Failed to load module \"dbe\" (invalid module, 0)\nto:\n (EE) Failed to load module \"dbe\" (loader failed, 7)\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Adam Jackson <ajax@redhat.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/e4dcf580f007d6adcf5b0c639865d7aaab1a319d"},{"author":{"email":"ea3248f8c2440b9439894eab21ef26209b22bacf@oracle.com","name":"Alan Coopersmith"},"sha":"c19c55a93ab1bc6db56075ca34d6e1c3f522f622","message":"Fix builds of Xnest & Xephyr with Solaris Studio compilers\n\nRequired in order to build with Studio cc now that xorg-macros is\nsetting -errwarn=E_FUNC_HAS_NO_RETURN_STMT since a bug in the Solaris\nsystem headers causes the noreturn attribute to not be correctly\napplied to the exit() prototype in <stdlib.h> when building with\nStudio instead of gcc.\n\nOtherwise compiler exits with errors:\n\"Display.c\", line 65: Function has no return statement : x_io_error_handler\n\"hostx.c\", line 341: Function has no return statement : x_io_error_handler\n\nUses Studio-specific pragma instead of adding another exit() prototype\nwith a noreturn attribute to avoid causing gcc to warn about having\na redundant prototype for the exit() function.\n\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>","url":"https://api.github.com/repos/shadchin/xserver/commits/c19c55a93ab1bc6db56075ca34d6e1c3f522f622"},{"author":{"email":"8a8cc541338200f67d4d0bcb65c78b6166649462@videotron.ca","name":"Gaetan Nadon"},"sha":"427cb53fddf0c517d1fd1025a87b1104735edd2a","message":"autoconf: remove redundant AC_PROG_MAKE_SET\n\nAlready included during Automake initialization.\n\nAfter the patch, no change:\nconfigure:3893: checking whether make sets $(MAKE)\nconfigure:3915: result: yes\n\nSigned-off-by: Gaetan Nadon <memsize@videotron.ca>\nReviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/427cb53fddf0c517d1fd1025a87b1104735edd2a"},{"author":{"email":"8a8cc541338200f67d4d0bcb65c78b6166649462@videotron.ca","name":"Gaetan Nadon"},"sha":"e89b0324da89ba5c0ba64af1ef46a12b7f55f879","message":"autoconf: the minimum required level for autoconf is 2.60\n\nThe toolchain requirements are documented here:\nhttp://www.x.org/wiki/ModularDevelopersGuide#Required_Tools\n\nNote that autoconf features only found in versions later\nthan 2.60 must not be used.\n\nSigned-off-by: Gaetan Nadon <memsize@videotron.ca>\nReviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nSigned-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/e89b0324da89ba5c0ba64af1ef46a12b7f55f879"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"6acebf9e1298939593b942ec91ae9ec9e74faa19","message":"include: add list_append()\n\nThe existing list_add() prepends to the list, but in some cases we need the\nlist ordered in the way we append the elements.\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/6acebf9e1298939593b942ec91ae9ec9e74faa19"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"c8e451a8e719ba432bcfa8976774c07307087809","message":"include: add list_last_entry to get the tail of a list\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/c8e451a8e719ba432bcfa8976774c07307087809"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"58dc73314b6508121ca094bbcf00612fe19ed69f","message":"include: point to the tests in list.c as examples\n\nEven with the documentation, the list.c tests are the best examples.\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/58dc73314b6508121ca094bbcf00612fe19ed69f"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"18539c89eca8f6e272ead2b631365da530065ae7","message":"include: a new list element does not need initialization, state so\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/18539c89eca8f6e272ead2b631365da530065ae7"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"7dfe8c32a96d3f96c8aaeb2802f5b122e381a1e4","message":"include: update struct list documentation to use one set of structs only\n\nThe example at the top of the file used a struct bar and a list of struct\nfoos. Use those two throughout instead of a different struct foo for the\nexamples and for the API documentation.\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/7dfe8c32a96d3f96c8aaeb2802f5b122e381a1e4"},{"author":{"email":"b9c50af7afa7642f6f854434cda9ffe54ece8b85@who-t.net","name":"Peter Hutterer"},"sha":"fac464e310b82fadcedf790798d1016c4805640b","message":"include: rename \"foos\" to \"list_of_foos\" in the struct list documentation\n\nMakes things a little easier to read.\n\nSigned-off-by: Peter Hutterer <peter.hutterer@who-t.net>\nReviewed-by: Chase Douglas <chase.douglas@canonical.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/fac464e310b82fadcedf790798d1016c4805640b"},{"author":{"email":"4482611cc290c2ce1399431891dcb9221c95f994@redhat.com","name":"Adam Jackson"},"sha":"22605effd188436629a0dbc688666549473741e4","message":"fbdevhw: iterate over all modes that match a mode. (v3)\n\nSo on RHEL5 anaconda sets an xorg.conf with a fixed 800x600 mode in it,\nwe run radeonfb and fbdev since ati won't work in userspace due to domain\nissues in the older codebase.\n\nOn certain pseries blades the built-in KVM can't accept an 800x600-43 mode,\nit requires the 800x600-60 mode, so we have to have the kernel radeonfb\ndriver reject the 800x600-43 mode when it sees it. However then fbdev\ndoesn't try any of the other 800x600 modes in the modelist, and we end up\ngetting a default 640x480 mode we don't want.\n\nThis patch changes the mode validation loop to continue on with the other modes\nthat match to find one that works.\n\nv2: move code around to avoid extra loop, after comment from Jamey.\nv3: move loop setup back into loop as per Jeremy's review.\n\nSigned-off-by: Dave Airlie <airlied@redhat.com>\nReviewed-by: Jamey Sharp <jamey@minilop.net>\nReviewed-by: Jeremy Huddleston <jeremyhu@apple.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/22605effd188436629a0dbc688666549473741e4"},{"author":{"email":"f2295d84e358395675bc8031be58672073ae065e@redhat.com","name":"Dave Airlie"},"sha":"b62dc4fcbcffd10de16650bee284702c8608bb60","message":"xext: don't free uninitialised pointer when malloc fails. (v2)\n\nInitialise the pAttr->values to values so if the values allocation\nfails it just ends up as free(NULL).\n\nPointed out by coverity.\n\nv2: use Alan's suggestion.\n\nSigned-off-by: Dave Airlie <airlied@redhat.com>\nReviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>\nReviewed-by: Jeremy Huddleston <jeremyhu@apple.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/b62dc4fcbcffd10de16650bee284702c8608bb60"},{"author":{"email":"f2295d84e358395675bc8031be58672073ae065e@redhat.com","name":"Dave Airlie"},"sha":"682c09a2cedd234b005334cc01247d859dd7f26a","message":"Xi: avoid overrun of callback array.\n\nThis code had an off-by-one and would allow writing one past the end of\nthe callbacks array.\n\nPointed out by coverity.\n\nSigned-off-by: Dave Airlie <airlied@redhat.com>\nReviewed-by: Jeremy Huddleston <jeremyhu@apple.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/682c09a2cedd234b005334cc01247d859dd7f26a"},{"author":{"email":"f2295d84e358395675bc8031be58672073ae065e@redhat.com","name":"Dave Airlie"},"sha":"1049139499d9132a20cd6d4d156fe9da9cddb6c2","message":"xaa: avoid possible freed pointer reuse in epilogue\n\nIf the pGCPriv->flags == 2, then we try to assign the freed pGCPriv->XAAOps\navoid this by clearing the flags in to be destroyed pGCPriv.\n\nReported by coverity.\n\nSigned-off-by: Dave Airlie <airlied@redhat.com>\nReviewed-by: Jeremy Huddleston <jeremyhu@apple.com>","url":"https://api.github.com/repos/shadchin/xserver/commits/1049139499d9132a20cd6d4d156fe9da9cddb6c2"}],"head":"372a6f10dc2d74d2d179e8b92449e9b8636a99ef","push_id":53452817,"size":45}