summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* configure.ac: bump to version 1.7.0 for release1.7.01.7Bryce Harrington2015-02-131-2/+2
|
* configure.ac: bump to version 1.6.93 for rc2 release1.6.93Bryce Harrington2015-02-061-1/+1
|
* configure.ac: Fallback to older detection code if pkg-config can't find expatBill Spitzak2015-02-061-1/+10
| | | | | | | | | | | | | | | This paritally reverts commit a4afd90f9f0c27ed5f3f313b915c260673f8be34. On older expat versions (ie the one on Ubuntu 12.04) there is no pkg-config file, so fall back to a test for the header and library. In addition the source for expat does not seem to be in a git repository but in cvs instead and it seems preferrable to not require cvs to build wayland. The restored test has been updated to use AC_SEARCH_LIBS. This version uses empty square brackets for the unused branches, similar to many other if statements in configure.ac. Reviewed-by: Bryce Harrington <bryce@osg.samsung.com> Reviewed-by: Andrew Oakley <aoakley@espial.com>
* tests: Fix FAIL in sanity-test (*timeout*) when Yama LSM enabledBryce Harrington2015-02-061-11/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a regression in the testsuite since c3653f7f, where four of the timeout tests fail with "Timeouts suppressed" messages. The timeouts are being suppressed because the testsuite is erroneously detecting that a debugger is attached. This detection mechanism (adopted from libinput) uses ptrace to test if there is a debugger parent process that can be attached. Unfortunately, this is an unreliable test: Kernel security policies exist to restrict the scope of ptrace to prevent processes from snooping on one another.[1] This security policy is set as the default on Ubuntu, and potentially other Linux distributions.[2] The Yama documentation suggests, "For software that has defined application-specific relationships between a debugging process and its inferior (crash handlers, etc), prctl(PR_SET_PTRACER, pid, ...) can be used. An inferior can declare which other process (and its descendents) are allowed to call PTRACE_ATTACH against it." This prctl call has no effect if Yama LSM is not loaded. The child needs to be synchronized to the client to prevent a race condition where the child might try to operate before the parent has finished its prctl call. This synchronization is done via pipes. This patch can be tested by running sanity-test with /proc/sys/kernel/yama/ptrace_scope set to 0 or 1; the test must pass for either value. 1: http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2d514487faf188938a4ee4fb3464eeecfbdcf8eb 2: https://wiki.ubuntu.com/SecurityTeam/Roadmap/KernelHardening#ptrace_Protection Signed-off-by: Bryce Harrington <bryce@osg.samsung.com> Reviewed-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com> Reviewed-by: Derek Foreman <derekf@osg.samsung.com> v4: Allow parent to communicate error state to child to prevent leaving child in zombie state if parent hits an error. v5: Check errno instead of rc for error. Don't waitpid on ppid. Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: Fill in high level description for SurfacesBryce Harrington2015-02-041-3/+10
| | | | | Signed-off-by: Bryce Harrington <bryce@osg.samsung.com> Reviewed-by: Jon A. Cruz <jonc@osg.samsung.com>
* test-runner: wait for concrete pidMarek Chalupa2015-02-041-1/+1
| | | | | | | | | | | | | After running a test in fork, we were waiting for any child to terminate. It is OK unless the child forks again. If the child calls fork, the waitid can catch the child's child termination, stop block and run another test while the former test is still running. This is racy i. e. when adding socket. Since we have test compositor which uses fork, this situation can occur pretty frequently. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Jon A. Cruz <jonc@osg.samsung.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* configure.ac: re-bump version to 1.6.92 for rc1 release1.6.92Bryce Harrington2015-01-301-1/+1
|
* doc: Fix out-of-tree build and also distcheckJon Cruz2015-01-302-2/+9
| | | | | | | | Corrects an issue that would cause out-of-tree builds to fail and also a few items that would cause distcheck to fail. Signed-off-by: Jon A. Cruz <jonc@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* configure.ac: bump version to 1.6.92 for rc1 releaseBryce Harrington2015-01-301-1/+1
|
* gitignore: Add the new cpp-compile-testBryce Harrington2015-01-291-0/+1
|
* doc: Intro text for doxygen output in it's own fileBill Spitzak2015-01-295-46/+155
| | | | | | | | | | | | | | | | | (This patch has been modified to apply atop current master) This makes it considerably easier to edit the text and make it different for each library. To address previous concerns with this patch, I wrote some more complete introductory text. This is based on my understanding of these libraries, which may not be correct, and is pretty rudimentary for libwayland-server! However this intro text demonstrates how to create links to the doxygen-generated text. It looks like you cannot link to methods easily as the link name contains a hash number, but links to objects and classes work. Reviewed-by: Jon A. Cruz <jonc@osg.samsung.com> Tested-by: Jon A. Cruz <jonc@osg.samsung.com>
* configure.ac: use pkg-config to find expatAndrew Oakley2015-01-282-10/+2
| | | | | | This is now done in the same way as the libffi dependency and still allows the library to be installed in a non-standard location (with PKG_CONFIG_PATH).
* doc/publican/Makefile.am: Add a missing order-only prerequisiteRui Matos2015-01-281-1/+1
| | | | | | | | Otherwise a parallel make invocation could fail due to the directory not existing. Signed-off-by: Rui Matos <tiagomatos@gmail.com> Reviewed-by: Jon A. Cruz <jonc@osg.samsung.com>
* event-loop: Dispatch idle callbacks twiceDerek Foreman2015-01-281-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | To fix a shutdown crash in weston's x11 compositor I want to move the weston X window close to an idle handler. Since idle handlers are processed at the start of an event loop, the handler that deals with window close will run at the start of the next input_loop dispatch, after which the dispatcher blocks on epoll forever (since all input events that will ever occur have been consumed). Dispatching idle callbacks both at the start and end of event-loop processing will prevent this permanent blocking. Note that just moving the callback dispatch could theoretically result in an idle callback being delayed indefinitely while waiting for epoll_wait() to complete. Callbacks are removed from the list when they're run, so the second dispatch won't result in any extra calls. Signed-off-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Giulio Camuffo <giuliocamuffo@gmail.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: update diagrams for compatibility.Jon Cruz2015-01-282-47/+47
| | | | | | Change attribute separators for compatiblity with graphviz older than 2.30. Signed-off-by: Jon A. Cruz <jonc@osg.samsung.com>
* doc: Create hot-linked areas in documents.Jon Cruz2015-01-284-35/+134
| | | | | | | Added xslt processing to give DocBook output diagram image maps/hot-linked areas consistent with those automatically generated by Doxygen. Signed-off-by: Jon A. Cruz <jonc@osg.samsung.com>
* doc: Switch from static image files to generated diagrams.Jon Cruz2015-01-287-13/+137
| | | | | | | Switches diagrams from using static PNG images to instead generate them via simple graphviz DOT markup files. Signed-off-by: Jon A. Cruz <jonc@osg.samsung.com>
* display-test: disable leak check in tests that use pthreadMarek Chalupa2015-01-281-0/+8
| | | | | | | pthread is leaking and it makes our tests fail. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* tests: add possibility to disable leak check for single testMarek Chalupa2015-01-283-0/+21
| | | | | | | | | | In tests that are using external libraries (i. e. pthread) we can get failure because of leaks in the external library. Until we have some better solution (if ever), let these (and only these) tests to disable leak checks. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* client: unref or destroy proxy when releasing queueMarek Chalupa2015-01-281-43/+60
| | | | | | | | When we release event queue with queued events, we can leak proxies in some cases. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* client: release display queue in wl_display_disconnect()Marek Chalupa2015-01-281-0/+1
| | | | | | | Don't leak events, not even on exit Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* connection-test: add tests for closure leaksMarek Chalupa2015-01-281-0/+79
| | | | | | | When we destroy closure, we are leaking memory sometimes. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* tests: add tests for leak check in clientsMarek Chalupa2015-01-282-0/+92
| | | | | | | | | Sanity tests for leak checks in clients of test compositor and also check if the test-compositor itself is not leaking anything. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* test-compositor: extend leak checks into clientsMarek Chalupa2015-01-281-0/+15
| | | | | | | | | | | | | | | | | | | Run leak checks also on clients of test-compositor. Checking leaks in memory is the same as in normal TEST. Checking file descriptor leaks is slightly more complex, as we pass an open file descriptor in the WAYLAND_SOCKET environment variable, which will be consumed by a client calling wl_display_connect(), but otherwise remain unused. We manage this accounting by checking if the environment variable has been unset by wl_display_connect() as it consumes the socket. Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com> [daniels: squashed counter-proposal patch to simplify fd leak checking, plus Bill's counter-counter-proposal, and reworded log.]
* test-runner: move leak checking into functionMarek Chalupa2015-01-282-19/+41
| | | | | | | | 1) now we can use it in the test-compositor 2) it looks better Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* scanner: Fix header generation for server protocolsMariusz Ceier2015-01-271-1/+1
| | | | | | | | | | Server protocols headers should include wayland-server.h, instead of wayland-util.h. Otherwise they're not useable with C++ compiler unless wayland-server.h was included earlier. Signed-off-by: Mariusz Ceier <mceier+wayland@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* tests: C++ compilation testMariusz Ceier2015-01-273-0/+25
| | | | | | | | | | | | | | | This test includes one of wayland headers, which produced error with C++ compiler. C compiler can't be used for this test, because it issues only a warning[1] and only when wayland headers are not installed in system headers path (/usr/include). [1] wayland-server-protocol.h:201:2: warning: implicit declaration of function ‘wl_resource_post_event’ [daniels: Merged in Marek's follow-up to check for a C++ compiler.] Signed-off-by: Mariusz Ceier <mceier+wayland@gmail.com> Signed-off-by: Marek Chalupa <mchqwerty@gmail.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
* tests: Correct return code handlingBryce Harrington2015-01-261-2/+2
| | | | | Reviewed-by: Marek Chalupa <mchqwerty@gmail.com> Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
* tests: Fix typo "evnironment"Bryce Harrington2015-01-261-1/+1
| | | | | Signed-off-by: Bryce Harrington <bryce@osg.samsung.com> Reviewed-by: Marek Chalupa <mchqwerty@gmail.com>
* doc: Put a dash between type/enum value and descriptionBill Spitzak2015-01-261-5/+5
| | | | | | This was suggested by Derek Foreman, I think it looks better Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: Add macros and typedefs to the documentationBill Spitzak2015-01-261-1/+2
| | | | | | | | | If somebody bothered to put a doxygen comment in for a macro or typedef, make it appear in the pages. This produces documentation for wl_container_of and wl_dispatcher_func_t from the _8h files. Reviewed-by: "Jon A. Cruz" <jonc@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: there is no need to move the _8h filesBill Spitzak2015-01-261-8/+0
| | | | | | | | The current xslt skips all the data that is in them, so it is ok if they are included. Reviewed-by: "Jon A. Cruz" <jonc@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* cosmetic: Cleanup trailing whitespaceBryce Harrington2015-01-261-15/+15
|
* cosmetic: Move the deprecated functions back to the end of the fileDerek Foreman2015-01-261-50/+55
| | | | | | | | | | There are functions below the "Deprecated functions below" comment that are not deprecated. Move the deprecated functions back down, and add a comment at the end of the file to try to keep this from happening again. Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
* doc: remove redundant subtitlesBill Spitzak2015-01-231-13/+25
| | | | | | | | | Put the argument lists next to the event/message title, which I think makes it a lot easier to understand, and remove redundant "values" title from enumerations. Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: Add object types and links to arguments int protocol documentationBill Spitzak2015-01-231-0/+35
| | | | | | | This makes it a lot easier to figure out what is going on! Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: compress the lists in the protocol docs someBill Spitzak2015-01-231-9/+21
| | | | | | | | Use simpara to remove the blank lines, and put the type/value and the comment into the same line. Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: made functions taking wl_event_queue arg belong to wl_event_queueBill Spitzak2015-01-231-4/+19
| | | | | | | | | | The fact that these functions take both a display and queue argument is I think historical, and they really are methods on the queue. Also added some docs for wl_display_prepare_read_queue. Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: Remove wl_map from documentationBill Spitzak2015-01-231-0/+4
| | | | | | | | This object is only in wayland-private.h so it's methods should not be in the documentation. Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: Remove deprecated functions from documentationBill Spitzak2015-01-231-1/+3
| | | | | Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* doc: add missing \memberof to wl_display_get_protocol_errorBill Spitzak2015-01-231-4/+3
| | | | | Reviewed-by: Derek Foreman <derekf@osg.samsung.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* server: Use existing id variable when inserting created objectJonas Ådahl2015-01-231-3/+2
| | | | | | | | We already have the id variable there and it makes it slightly easier to read. Signed-off-by: Jonas Ådahl <jadahl@gmail.com> Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
* protocol: Clarify selection data offer destruction requirementJonas Ådahl2015-01-231-1/+3
| | | | | | | | Clarify that a client receiving a wl_data_device.selection event must destroy the data_offer of the previous wl_data_device.selection event, if any. Signed-off-by: Jonas Ådahl <jadahl@gmail.com>
* protocol: Fix typo in wl_data_offerJonas Ådahl2015-01-231-1/+1
| | | | Signed-off-by: Jonas Ådahl <jadahl@gmail.com>
* configure.ac: bump version to 1.6.91 for the alpha release1.6.91Bryce Harrington2015-01-171-1/+1
|
* doc: make itemized lists from doxygen workBill Spitzak2014-12-191-0/+8
| | | | | | Not actually used currently but probably a good idea. Acked-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
* doc: Document structures and unions in addition to classesBill Spitzak2014-12-191-10/+15
| | | | | | | | | This provides targets for some of the doxygen links, and some of them have useful memberof function lists. Added some if/else statements to reduce validation errors. Tested-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
* doc: Each class in doxygen output is a sectionBill Spitzak2014-12-191-15/+21
| | | | | | | | | | All the methods belonging to the class are listed with it, making it much easier to find them. I dumped all other functions into a section called "Functions" at the end. Tested-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
* doc: removed the word "interface" from the link namesBill Spitzak2014-12-194-21/+18
| | | | | | | | Just to make it slightly shorter. Also add a dash to the doxygen links to make them look a bit more alike. Tested-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
* doc: Reduce the validation errors of the docbook inputBill Spitzak2014-12-193-33/+23
| | | | | | | | | | | | | | | | | | | | | | (this is different from previous version as it removes some broken and irrelevant changes to the protocol appendix). This removes all the validation errors except for missing link targets. You can test this by removing the --skip-validation from doc/publican/Makefile.am. Main changes are to avoid nesting <para> commands. I also used <simpara> in some places to reduce the amount of blank space. And the reference id's are prefixed with the chapter name to avoid collisions between libclient and libserver. PS: it would be useful if somebody who actually knows something about xslt would come up with a way to translate a block of text makde of <para> commands unchanged, but add <para> around plain text. Most of the difficulty is that doxygen's output is rather inconsistent here. Tested-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>