Only update the surface size if a new buffer is attached

This fixes a regression caused by either 918f2dd4 or da75ee1d.  In
particular, if a client called commit without attaching a buffer and if the
compositor had already released its reference to the buffer, then a size of
0x0 would be set on the surface.  In particular, this affects the wayland
backend because it frequently sends only a frame request in order to cause
a refresh.

Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Fix XWayland transient window location.

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

Signed-off-by: Axel Davy <axel.davy@ens.fr>
clipboard: check for NULL malloc result

If malloc fails to allocate memory for a clipboard_client, close
the file descriptor (fd) and return.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
gl-renderer: check for NULL malloc result

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
backlight: fix backlight_path memory leak

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
log: print dummy timestamp if localtime call fails

Handle the case where localtime fails (NULL) and print
something else to indicate localtime is erroneous.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
wcap-decode: free malloc'd frame variable

The 'frame' variable does not appear to be used for anything
useful and is probably dead code... perhaps we should just remove
it.

Nonetheless, at least free it at the end of the program.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
wcap-decode: check for NULL results

Check for malloc failures and failed wcap_decoder_create.
Print context messages to stderr when things fail.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
wcap-decode: remove dead code

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
screenshooter: check for NULL malloc and fix memleak

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
clients/desktop-shell: check OOM conditions

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
use xzalloc wrapper instead

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
terminal: use xstrdup instead to handle OOM

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
weston-info: free allocated memory and check OOM conditions

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
clients/window: fail if NULL returned from frame_create

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
exposay: don't try to activate if esurface is NULL

Fix logic so view can't be NULL deref'd... that is, previously
if 'esurface' was NULL, then 'view' remained NULL and we
attempted to deref 'view' unconditionally.  Instead, if
'esurface' is NULL then we can't get the view so just return
immediately.

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
input-panel: validate return value of get_default_view

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
shell: validate get_animation_type parameter

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>
shell: validate create_focus_surface results

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>