GitLab

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Resolves issues reported by Oracle Parfait static analyzer:

Error: Memory leak
   Memory leak [memory-leak] (CWE 401):
      Memory leak of pointer pixels allocated with malloc((pScrn->videoRam * 1024))
        at line 803 of dummy_driver.c in function 'DUMMYScreenInit'.
          pixels allocated at line 790 with malloc((pScrn->videoRam * 1024))
   Memory leak [memory-leak] (CWE 401):
      Memory leak of pointer pixels allocated with malloc((pScrn->videoRam * 1024))
        at line 805 of dummy_driver.c in function 'DUMMYScreenInit'.
          pixels allocated at line 790 with malloc((pScrn->videoRam * 1024))
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Matt Turner <mattst88@gmail.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Found by using:
    codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Marek Marczykowski-Górecki authored Sep 27, 2016 and Adam Jackson committed Mar 10, 2020

Apparently 4 is too low. The number is totally arbitrary.
Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>

Marek Marczykowski-Górecki authored Jul 22, 2016 and Adam Jackson committed Mar 10, 2020

Add WIDTH_MM and HEIGHT_MM output properties to allow setting "physical"
size of the output. Normally this data is retrieved from EDID, but for
obvious reasons it can't be done in the VM.
This patch allows to set this properties from user space. This is later
reported as output size (the header line describing the output in
`xrandr` output).
Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>

Marek Marczykowski-Górecki authored Oct 16, 2018 and Adam Jackson committed Mar 10, 2020

Properly initialize RandR extension, mostly with stubs. It creates four
virtual outputs (lets hope that nobody will need more at once...). By
default only the first one is in "connected" state, with preferred mode
from xorg.conf. Other outputs get "connected" at first mode set (and
left connected forever - you can always not use it).
So without any xrandr calls it should work as before.
Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Antoine Martin authored Aug 14, 2017 and Adam Jackson committed Aug 14, 2017

This patch makes it possible to start the server using the dummy driver
with a 30 bit depth.  The colormap size is changed from 256 to 1024 to
prevent crashes.
Reviewed-by: Adam Jackson <ajax@redhat.com>

GCC's -Wmisleading-indentation complains about this

  dummy_driver.c: In function ‘DUMMYGetRec’:
  dummy_driver.c:181:5: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
       if (pScrn->driverPrivate == NULL)
       ^~
  dummy_driver.c:183:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘if’
           return TRUE;
           ^~~~~~
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>

Xiaolei Yu authored Aug 05, 2017 and Aaron Plattner committed Aug 05, 2017

Signed-off-by: Xiaolei Yu <dreifachstein@gmail.com>
Reviewed-by: Antoine Martin <antoine@nagafix.co.uk>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>

The pointer to the screen's pixels is stored in the screen pixmap's
devPrivate.ptr field.  Just use that instead of storing it
redundantly.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Reviewed-by: Bob Terek <xorg@esoterek.com>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>

DGA was so 2004.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Reviewed-by: Bob Terek <xorg@esoterek.com>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>

This is assigned, but never used.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Reviewed-by: Bob Terek <xorg@esoterek.com>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>

As far as I can tell, this was never implemented or used.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Reviewed-by: Bob Terek <xorg@esoterek.com>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>

Mihail Konev authored Jan 26, 2017 and Peter Hutterer committed Jan 26, 2017

Signed-off-by: Mihail Konev <k.mvc@ya.ru>

Emil Velikov authored Mar 09, 2015 and Peter Hutterer committed Jan 26, 2017

Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
fall-outs, when they contain space.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Syncs the invocation of configure with the one from the server.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

Signed-off-by: Julien Cristau <jcristau@debian.org>

dixChangeWindowProperty was introduced in that release.
Signed-off-by: Julien Cristau <jcristau@debian.org>

These functions might be useful in a real driver, but with no
hardware, they're pointless.  Get rid of them.

v2: Rebase, get rid of pointless calls to DUMMYAdjustFrame, return TRUE from
DUMMYSwitchMode.
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>
Reviewed-by: Antoine Martin <antoine@nagafix.co.uk>
Tested-by: Antoine Martin <antoine@nagafix.co.uk>

Antoine Martin authored Sep 20, 2016 and Aaron Plattner committed Sep 21, 2016

Signed-off-by: Antoine Martin <antoine@nagafix.co.uk>
Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com>
Reviewed-by: Aaron Plattner <aplattner@nvidia.com>
Signed-off-by: Aaron Plattner <aplattner@nvidia.com>

eb36924ead40564325aa56d54a973dc8fb4eae83 removed ChangeWindowProperty from the
server.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Jon Turney <jon.turney@dronecode.org.uk>

Antoine Martin authored Sep 17, 2015 and Adam Jackson committed Sep 17, 2015

Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Antoine Martin <antoine@nagafix.co.uk>

See http://people.gnome.org/~walters/docs/build-api.txt

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Julien Cristau <jcristau@debian.org>

Signed-off-by: Adam Jackson <ajax@redhat.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

Generated from util/modular/x-driver-screen-scrn-conv.sh
Signed-off-by: Dave Airlie <airlied@redhat.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

This affects only Cygwin, where drivers must be linked against the
Xorg implib.  On other systems, XORG_LIBS will be empty.
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

If our server supports DGA and we want to build the dummy driver without it,
XFreeXDGA will be defined by the server and will be picked up rather than
our configuration option.  This change forces us to honor our configuration
hoice.
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

Signed-off-by: Cyril Brulebois <kibi@debian.org>

Jeremy Huddleston Sequoia authored Sep 14, 2011 and Jamey Sharp committed Sep 14, 2011

Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Jamey Sharp <jamey@minilop.net>