malloc + memset -> zalloc

And for clients using the xmalloc helper, use xzalloc.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add zalloc(size_t) allocator function

Same as calloc(1, len).

cms-colord: Don't redefine _GNU_SOURCE

warning: "_GNU_SOURCE" redefined [enabled by default]

See commit c228e23b

clients: use xmalloc in more places

For the clients continue to use xmalloc() to simplify OOM-handling.

Signed-off-by: Brian Lovin <brian.j.lovin@intel.com>

gl-renderer: Only check for subimage when we have extension at compile time

If weston is compiled against a gl2ext.h that doesn't have the subimage
extension, but then run against a gles2 library that does provide it,
we end up disabling the glTexImage2D falback without having the subimage
code paths compiled in.

Add workaround for broken GL_EXT_unpack_subimage tokens

Earlier versions of gl2ext.h defined the GL_EXT_unpack_subimage tokens
without the _EXT suffix.  That's been fixed and we're using the _EXT
tokens now, but just in case we're using an older, broken header, define
the _EXT tokens manually.

gl-renderer.c: Cast wl_resource to void * to avoid warning

The extension has been updated to take a wl_resource * now, but to avoid
warnings when compiling against older versions, just cast to void *.

gl-renderer: Test for GL_EXT_unpack_subimage not GL_UNPACK_ROW_LENGTH

It is defined by the mesa #include, which is just a copy of the official
Khronos header.  It's just defined in a different section than the
extension tokens.  In the mean time, the extension tokens were renamed
to add a _EXT suffix (eg GL_UNPACK_ROW_LENGTH -> GL_UNPACK_ROW_LENGTH_EXT)
and we silently failed to used the subimage extension.

nested: Update to use weston-nested-client binary name

Use #define WL_HIDE_DEPRECATED to avoid deprecated warnings for wl_buffer

clients: Fix typo in simple-touch and simple-shm.

listenter -> listener. Better fix it now before it spreads further.

Signed-off-by: Stefan Schmidt <s.schmidt@samsung.com>

Drop extraneous duplicate header includes

Signed-off-by: Bryce Harrington <b.harrington@samsung.com>

rpi: Add support for SHM buffers in RGB565 format

pixman-renderer: Add support for SHM buffers in RGB565 format

gl-renderer: Add support for SHM buffers in RGB565 format

window: Allow hinting of a preference for RGB565 when creating a window

And check if the renderer supports the RGB565 format for wl_shm buffers
before creating the cairo surface and requesting the buffer.

It can save quite some memory with big surfaces such as desktop
backgrounds.

compositor: Check wl_resource_create() return value

This fixes a number of call-sites to properly check for NULL and return
the no memory event when allocation fail.

data-device.c: Handle failure to allocate offer in weston_drag_set_focus

If we fail to allocate and send the offer, don't send the drag enter and
don't set the drag focus so we don't send motion events or leave.

data-device.c: Handle OOM in weston_data_source_send_offer()

If we fail to allocate the resource, clean up and return NULL.

desktop-shell: Refactor launcher configuration reading to avoid leaking

compositor-x11: Don't leak the looked up name for skipped outputs

window: Free the allocated display if we fail to setup libxkbcommon

tablet-shell: Avoid leaking the path on failed icon loading

image: Free filename saved into structure on error path

compositor-drm: Use a format width parameter for the modeline sscanf

compositor-fbdev: Close fd used for re-enabling if that fails

The device will be opened again in fbdev_output_create().

compositor-fbdev: Avoid dereferencing a pointer in freed memory

fbdev_output_destroy will free the memory passed into in and since we
want to pass the device name into fbdev_output_create we need to save
this to an intermediate value

tty: Correctly check if the opening of the file descriptor failed

wcap-decode: Close file descriptor when destroying the decoder

drm: close the drm file descriptor when the compositor is destroyed

autotools: Correctly prefix simple-egl client name

weston-launch: Fix signal mask typo

weston-launch: Unblock our signalfd signals before execing child

Giovanni Campagna ran into this with mutter.

clipboard: remove the weston_seat destruction listener on destroy

Prior to freeing the memory in which the link node for the signal is
emedded we should remove the link node from the list to prevent the list
from being corrupted.

https://bugs.freedesktop.org/show_bug.cgi?id=67231

text-backend: remove the weston_seat destruction listener on destroy

Prior to freeing the memory in which the link node for the signal is
emedded we should remove the link node from the list to prevent the list
from being corrupted.

https://bugs.freedesktop.org/show_bug.cgi?id=67231

screenshot: Drop extraneous 'is' to improve warning msg grammar

Signed-off-by: Bryce Harrington <b.harrington@samsung.com>

Reset touch focus in unmap

Otherwise, there will be race condition of visiting invalid surface data.

xwayland: Remove transform listener when destroying the wm

Fix a segfault occuring after the last X window was closed.

autotools: Add an option to install demo clients

This patch adds a configure option which will enable
user to install demo clients if desired. It is disabled
by default.

v2: Remove AC_DEFINE as it is not necesary

autotools: Rename demo clients

Prefix demo clients names with weston-, so they can
be installed if user desires.

evdev: Suport old-style single-touch touch screens

These screens send only BTN_TOUCH and ABS_X/Y.

evdev: Use temporary x and y coordinates when applying calibration

Don't overwrite device->abs.x halfway through the matrix multiplication.

evdev: Don't add output offset in evdev_process_absolute_motion()

We do that in weston_output_transform_coordinate() now.

window.c: Simplify window_create()

Use xmalloc in desktop-shell.c

compositor-x11: Use assert(0) instead of return NULL for non-reachable code

window.c: Use xmalloc for frame_create()

cliptest: Use xmalloc in cliptest

clipboard: Fix fd leak

We never closed the reading end of the pipe after copying data to the
clipboard.

Rename wl_data_offer to weston_data_offer

Another left-over from when we moved the input structs.

udev: Use WL_OUTPUT udev attribute for pairing touchscreens to outputs

weston-launch: Switch to regular poll

epoll doesn't provide any convenince here and is just awkward to use.

weston-launch: Don't start new session unless -u is given

weston-launch: Drop sleep_fork option

weston-launch: Move child setup and exec to its own function

Rename wl_data_source to weston_data_source

Missed this rename when we moved the input structs over from wayland.

clipboard: Handle OOM in clipboard_source_create()

clients: Introduce xmalloc() and use it a few places

For the sample clients we introduce xmalloc() to simplify OOM-handling.
This patch only converts a few callsites, but this will be our strategy
going forward.

shared: Add out-of-memory handling to theme_create()

compositor: Move weston-launch socket lookup to drm backend

This is the only backend that uses the launcher, so let's set it
up there.

shell: Remove redundant weston_pointer field from shell_grab

This is already in weston_grab.

evdev: Use weston_output_transform_coordinate for evdev touch screens

compositor: Move device coordinate transform helper to compositor.c

This function transform input coordinates from output space to
compositor space and is useful for X input as well as touch screen input.

simple-touch: Attach buffer before posting damage

weston-launch: Ignore sighup

When our session terminates all processes get a SIGHUP.  We need to ignore
that in weston-launch and stay around to help weston shut down properly.

https://bugs.freedesktop.org/show_bug.cgi?id=63004

Bump version to 1.2.0

window: Disregard motion events outside our current surface dimensions

It is possible to receive a motion event that was generated by the
compositor based on a pick of a surface of old dimensions. This was
triggerable on toytoolkit clients when minimising. The new window
dimensions were propagated through the widget hierarchy before the event
was dispatched.

This issue was triggering a segfault due to the focussed widget being
lost as the client code tried to identify which widget should have the
focus using co-ordinates outside the dimensions of the surface.

https://bugs.freedesktop.org/show_bug.cgi?id=66795

editor: Support shift-left/right for selecting text

If the shift modifier is active then we don't make the cursor and the
anchor the same and as a result we develop a selection in the direction
that the arrow key gets pressed in.

https://bugs.freedesktop.org/show_bug.cgi?id=66802

shell: Respect output transformation for input panel surface

Rather than using the dimensions in the mode we can use the recently
added output width and height members which are updated to reflect any
output rotation.

https://bugs.freedesktop.org/show_bug.cgi?id=66798

window: Allow popup menu when the window is maximised

https://bugs.freedesktop.org/show_bug.cgi?id=66793

wcap: Fix README typo

configure.ac: Set automake conditional after testing for colord pkg

Bump versionto 1.1.91

clients/editor.c: Draw cursor at correct verical position

With this patch, the editor cursor now moves up and down correctly
instead of having the top part stuck on the first line.

clients/editor.c: Ignore ESC key

Otherwise, editor would print the "unknown UTF-8 glyph" boxes.

tablet-shell: Avoid infinite loop when unlocking

weston_compositor_wake will fire the signal that the unlock handler is
setup as the listener for. Instead lets change the state to HOME which
unlocks.

tablet-shell: Fix copy and paste error in unlock handler

The signal handler was using the wrong member to find the containing
structure of the unlock listener.

https://bugs.freedesktop.org/show_bug.cgi?id=57637

Convert to wl_global_create/destroy()

tests: Fix warnings in config-parser-test

tests: Move config-parser.test to tests/

Signed-off-by: Quentin Glidic <sardemff7+git@sardemff7.net>

Use the average width of ASCII glyphs for cell width

As some CJK fonts are dual-width, calculate the average width of ASCII
glyphs and use that instead of the max_x_advance of the font.  This is
what VTE does too.

https://bugs.freedesktop.org/show_bug.cgi?id=63796

input: Remove wl_seat global when a seat is destroyed

The input code was relying on compositor destruction to clean up the
global, but that doesn't work when the global comes and goes dynamically.

https://bugs.freedesktop.org/show_bug.cgi?id=65913

xwayland: Don't allow resizing to 0x0 or less

X windows have to be at least 1x1 pixels.

https://bugs.freedesktop.org/show_bug.cgi?id=63510

weston-launch: Fix the typo for real this time

Thanks to MonkeyofDoom for point this out.  I'm still adjusting to the
fact that there's no 'd' in privilege, but I'll get over it.

compositor: Eliminate marshalling warning for leave events

Don't NULL the resource pointer before calling weston_surface_destroy().
We use to have more of a distinction between compositor created surfaces
and client surfaces, and weston_surface_destroy couldn't be used for
client surfaces.  Now it all goes through weston_surface_destroy() and
we can remove the assert and the NULL-ing of resource, which caused the
marshalling warning.

input: check if the resource is valid in seat_get_pointer

seat->pointer->focus->resource can be NULL, if the surface was
created with weston_surface_create.

Document RDP compositor options

Help was missing for the RDP compositor, this patch gives details
about available options.

Fix compilation with FreeRDP master

context_size has been renamed to ContextSize, this patch fixes it

tests: Add .weston extension to clients tests

We can then add tests which do not use Weston in the test suite.

Signed-off-by: Quentin Glidic <sardemff7+git@sardemff7.net>

weston-launch: Fix an error message typo

weston-launch: Set all groups for user

Signed-off-by: Quentin Glidic <sardemff7+git@sardemff7.net>
https://bugs.freedesktop.org/show_bug.cgi?id=65933

compositor: rebuild the global list if we've removed a surface from it

The list of surfaces used by weston_compositor_pick_surface() is
maintained in list of surfaces stored on the compositor. This list is
generated from the surfaces across all the layers using
weston_compositor_build_surface_list.

When destroying a surface the surface is "unmapped" with
weston_surface_unmap which removes it from the layer list. However since
the compositor surface list was only being rebuilt when the output was
repainted a call to weston_compositor_pick_surface before the next
output repaint would use an outdated surface list containing surfaces
that have been partially destroyed.

https://bugs.freedesktop.org/show_bug.cgi?id=65986
https://bugs.freedesktop.org/show_bug.cgi?id=66173
https://bugs.freedesktop.org/show_bug.cgi?id=66198

desktop-shell: Don't crash on output hotplug

The panel and background were never created for hotplugged outputs and
since some parts of the code assume that they always exist that would
lead to desktop-shell client to crash in that case.

This was easier to spot when the display was locked, because Weston
respawns the shell client and the user might not notice since there is
no flicker.

https://bugs.freedesktop.org/show_bug.cgi?id=66531

desktop-shell: Adapt to the new config parser API

This lets the code for adding panel launchers and setting up the
background to be moved into panel_* and background_* functions.

Note that this changes the behavior of the default launcher. Before
this change a default launcher would be added only if there was no
config file. Now a launcher is also added if there is no valid
launcher section.

compositor: Remove a plane from the compositor list when releasing it

Weston would crash when hot plugging/unplugging outputs using the DRM
backend due to a corrupted plane list.

https://bugs.freedesktop.org/show_bug.cgi?id=66530
https://bugs.freedesktop.org/show_bug.cgi?id=66529

tests: Rename xwayland_test to xwayland-test

Consitency.

xwayland: Don't crash when we get configure notify for destroyed frame windows

We can get a destroy notify for the frame window after we've removed it
from the hash table.  This turns into a NULL pointer deref when we look up
the window and try to use it for debugging printout.

Fixes the failing xwayland test case.

shell: Always go through end_busy_cursor when ending the busy cursor grab

input: Don't cache pointer grab interface between calls to focus and motion

The focus callback for the current grab can change the grab, so we have
to make sure we call the motion callback for the currently active grab.

https://bugs.freedesktop.org/show_bug.cgi?id=65961

shell: Dont set grab cursor if desktop-shell client died

If we don't have a desktop-shell helper client, don't try to send events
to it.

https://bugs.freedesktop.org/show_bug.cgi?id=65961