debian-bugs-dist@lists.debian.org

2024-01-09 Thread Alan Coopersmith

On Tue, 9 Jan 2024 19:40:30 +0100 =?utf-8?B?0L3QsNCx?= 
 wrote:

Package: xserver-xorg-input-libinput
Version: 1.3.0-1
Severity: normal
File: /usr/share/man/man4/libinput.4.gz

do notice Option ccelStepFallback "float" and similar, clearly mangled.


I believe this is already fixed upstream by:
https://gitlab.freedesktop.org/xorg/driver/xf86-input-libinput/-/merge_requests/45/diffs
which was included in the upstream 1.4.0 release.

--
    -Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#173314: xcalc only accepts digits from the numeric pad

2023-05-20 Thread Alan Coopersmith

This should be fixed now in the new upstream xcalc-1.1.2 release.

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#1000008: openscap: depends on obsolete pcre3 library

2023-04-15 Thread Alan Coopersmith

Upstream bug: https://github.com/OpenSCAP/openscap/issues/1873



Bug#1000012: nmap: depends on obsolete pcre3 library

2023-04-15 Thread Alan Coopersmith

Upstream bug: https://github.com/nmap/nmap/issues/1335



Bug#101937: [xwd] 'xwd -name windowtitle -frame' doesn't capture frame

2023-02-18 Thread Alan Coopersmith

This is operating as documented - in the xwd man page, it states:

   -frame  This  option  indicates that the window manager frame should be
   included when manually selecting a window.

But using -name is selecting a window automatically, not waiting for the user
to manually click on it.

With the current implementation, you would instead need to do:

"xwininfo -name ... -children | grep Parent"

and use the id returned by that with the -id argument to xwd:

"xwd -id ... -out image"

Making -frame work with the automatic selection arguments (-id & -name)
might be a nice enhancement, but it's not a bug that it doesn't yet.

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#201994: [xconsole] want mouse wheel to scroll log window

2023-02-18 Thread Alan Coopersmith
Previously translations for buttons 4 & 5 were only set for the scrollbar, so 
required the mouse to be over the scrollbar to use the mouse wheel.


I've submitted a merge request upstream to handle buttons 4 & 5 in the text
area as well:
https://gitlab.freedesktop.org/xorg/app/xconsole/-/merge_requests/7

You should be able to edit the XConsole app-defaults file on your system
to do the same until a new release is published.

--
    -Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#173314: xcalc only accepts digits from the numeric pad

2023-02-18 Thread Alan Coopersmith

This is because the key mappings in the translation list in app-defaults
only list the main set of number keys with a modifier of "None", so they
don't work when NumLock is active.  We can't remove that modifier since
then holding down shift and hitting one of them would enter the number
instead of doing the intended operation (!, *, (, or ) for instance).

Instead I've submitted a merge request upstream to add translations for
those keys with Mod2 (aka NumLock) in the modifier list - you should be
able to edit the XCalc app-defaults file on your system to do the same
until a new release is published:
https://gitlab.freedesktop.org/xorg/app/xcalc/-/merge_requests/11

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#92706: xcalc support for comma on numeric keypad

2023-02-18 Thread Alan Coopersmith

I believe this was fixed upstream a couple years after this bug was reported:

 341. Added KP_Separator to symbols that can generate decimal() in xcalc
  (Egbert Eich).

https://cgit.freedesktop.org/~libv/xfree86/commit/programs/xcalc/XCalc.ad?id=b4898b6e916dbc0f71daa5edaba1cc115ae5794d

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#720266: x11-apps: xman aborts closing search window

2023-02-18 Thread Alan Coopersmith

This is because xman was not installing a WM_DELETE_WINDOW handler for the
search window, so window managers fall back to XKillClient() to shut down
the entire program when the close button is clicked.

Upstream fix: https://gitlab.freedesktop.org/xorg/app/xman/-/merge_requests/6

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#742356: xkbprint(1): Some (typographical) corrections to the manual

2022-04-21 Thread Alan Coopersmith

Fixed upstream in xkbprint 1.0.5 by
https://gitlab.freedesktop.org/xorg/app/xkbprint/-/commit/b968db6ca6fdb6ebb2384eaffaabe406c5cf783f

--
-Alan Coopersmith- alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/solaris



Bug#726962: Bug #726962

2019-03-10 Thread Alan Coopersmith

The '..' macro was removed upstream in the xinit 1.4.0 release:
https://gitlab.freedesktop.org/xorg/app/xinit/commit/52689362

The hair spaces between dashes recommended in this bug were added upstream
in the xinit 1.4.1 release:
https://gitlab.freedesktop.org/xorg/app/xinit/commit/752ef176

The issue in the previous comment about the Debian text accidentally
inserted in the #ifdef __SCOMAN__ section appears have to been resolved
by reworking that patch for the 1.4.0 release after the __SCOMAN__
ifdef was removed upstream.

--
    -Alan Coopersmith-   alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/alanc



Bug#648158: Fix applied upstream

2019-02-17 Thread Alan Coopersmith

https://gitlab.freedesktop.org/xorg/app/xcompmgr/commit/9c86c0f21b9d34c0ae491327482415a946102c4f



Bug#875311: Bug #875311: gedit: CVE-2017-14108: CPU consumption via crafted file

2018-05-27 Thread Alan Coopersmith
Upstream bug: https://bugzilla.gnome.org/show_bug.cgi?id=791037



Bug#729812: Bug #729812, [libx11-doc] incorrect description of XkbGetNamedIndicator in man pages

2017-08-06 Thread Alan Coopersmith
Fixed upstream:
https://cgit.freedesktop.org/xorg/lib/libX11/commit/?id=c6dadd4cebd994aafb37a58b3adbaa82507c2d18

-- 
-Alan Coopersmith-   alan.coopersm...@oracle.com
 Oracle Solaris Engineering - https://blogs.oracle.com/alanc



Bug#761628: please do not use &fullrelvers; in xdmcp.xml

2016-01-01 Thread Alan Coopersmith

On 11/11/15 01:23 PM, Helmut Grohne wrote:

Hello Xorg developers,

While furthering Debian's multiarch capabilities I ran into the issue
that an xdmcp.txt generated from xdmcp.xml (in libxdmcp) would be
dependent on the version of xorg-sgml-doctools. Due to the
implementation of multiarch, the difference in file content breaks the
installation of libxdmcp. I am therefore proposing to drop the use of
&fullrelvers; in xdmcp.xml (see attached patch). Despite being motivated
by a technical constraint in Debian, this change makes sense
conceptually as well. The value of &fullrelvers; is independent of the
version of libxdmcp. I would not expect the version of a different tool
to show up in the header of the documentation for libxdmcp. I hope that
this makes sense to you as well. Please Cc me in your replies.


This is leftover from the old X11R6 monolithic releases, when everything
was released as one giant wad, and the &fullrelvers; was the version of
that wad.  It's stayed around mainly because we've had no reason to drop
it until now.

Since we've not even done a X11R7 katamari release in years, it's pretty
meaningless today, so I've pushed your suggested fix to git master for
libXdmcp:

To ssh://git.freedesktop.org/git/xorg/lib/libXdmcp
   b10f382..5021b93  master -> master

Thank you for your patch.

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc



Bug#749120: [PATCH libxtrans] Make sure LONG64 is defined if necessary

2015-09-24 Thread Alan Coopersmith

On 07/29/15 11:41 AM, Julien Cristau wrote:

Xtrans.h defines BytesReadable_t as int or long depending on whether
LONG64 is defined.  We need to make sure Xmd.h is included so our idea
of BytesReadable_t is consistent across compilation units.

Debian bug#749120

Reported-by: Michael Tautschnig 
Signed-off-by: Julien Cristau 
---
  Xtrans.h | 1 +
  1 file changed, 1 insertion(+)

diff --git a/Xtrans.h b/Xtrans.h
index 026fbd7..22e0440 100644
--- a/Xtrans.h
+++ b/Xtrans.h
@@ -52,6 +52,7 @@ from The Open Group.

  #include 
  #include 
+#include 

  #ifndef WIN32
  #include 


Reviewed-by: Alan Coopersmith 

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc



Bug#749120: [PATCH libFS] include X11/Xmd.h in fs_transport

2015-07-26 Thread Alan Coopersmith

On 07/26/15 11:16 AM, Julien Cristau wrote:

Xtrans defines BytesReadable_t as int or long depending on whether
LONG64 is defined.  So on a 64bit build it would be int when building
FSConnServ.c, but long for fs_transport.c.  To fix this inconsistency,
make fs_transport.c include .

Debian bug#749120

Reported-by: Michael Tautschnig 
Signed-off-by: Julien Cristau 
---
  src/fs_transport.c | 1 +
  1 file changed, 1 insertion(+)

I'm not sure if this should be fixed in xtrans instead...


Yeah - I think we should add the #include  to Xtrans.h, not
force every consumer to add it before including Xtrans.h.

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#726041: [PATCH] mi: fix typo in warning about overflowing queue

2015-07-02 Thread Alan Coopersmith

On 07/ 2/15 11:16 AM, Julien Cristau wrote:

Debian bug#726041

Reported-by: Vincent Hobeïka 
Signed-off-by: Julien Cristau 
---
  mi/mieq.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mi/mieq.c b/mi/mieq.c
index 8907a6e..b7be193 100644
--- a/mi/mieq.c
+++ b/mi/mieq.c
@@ -599,7 +599,7 @@ mieqProcessInputEvents(void)
  ErrorF("[mi] EQ processing has resumed after %lu dropped events.\n",
 (unsigned long) miEventQueue.dropped);
  ErrorF
-("[mi] This may be caused my a misbehaving driver monopolizing the 
server's resources.\n");
+("[mi] This may be caused by a misbehaving driver monopolizing the 
server's resources.\n");
  miEventQueue.dropped = 0;
  }


Heh, it only took a minute of starting to see the subtle change of "my" to "by".

Reviewed-by: Alan Coopersmith 

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#718682: liblcms1: Buffer overflows in Little CMS v1.19

2013-08-05 Thread Alan Coopersmith

On 08/ 5/13 04:35 PM, Alan Coopersmith wrote:

+fgets(Buffer, (Buffer_size - 1), stdin);
+sscanf(Buffer,"%s");


Oops, forgot to mention the sscanf is still wrong in this second revision.
This code now reads a line from stdin and writes it to Buffer.  The sscanf
now takes Buffer as input, looks for a string pattern matching %s and writes
it to, well, whatever the random uninitialized value is next on the stack,
because there is no output argument provided for the %s.   Fortunately,
gcc -Wformat should find this and error out before anyone ships it.

Of course, since this is used in a function that expects the string to be
returned in Buffer, having sscanf read from Buffer and write the result
somewhere else is also counterproductive.

--
    -Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#718682: liblcms1: Buffer overflows in Little CMS v1.19

2013-08-05 Thread Alan Coopersmith

On 08/ 5/13 05:00 PM, Pedro Ribeiro wrote:

Thanks for that Alan - I had no idea, and have been looking at lots of C code
lately that has probably has the same mistakes. I will keep an eye on that.


More details/deeper explanations of when sizeof can and cannot work can be
found in:

https://www.securecoding.cert.org/confluence/display/seccode/ARR01-C.+Do+not+apply+the+sizeof+operator+to+a+pointer+when+taking+the+size+of+an+array

https://en.wikibooks.org/wiki/C_Programming/Pointers_and_arrays#sizeof

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#718682: liblcms1: Buffer overflows in Little CMS v1.19

2013-08-05 Thread Alan Coopersmith

 void GetLine(char* Buffer)
 {
-scanf("%s", Buffer);
+size_t Buffer_size = sizeof(Buffer);
+fgets(Buffer, (Buffer_size - 1), stdin);
+sscanf(Buffer,"%s");


sizeof() in the C language does not reach through a pointer to find the size of
the underlying object - that code will always set Buffer_size to the size of
the pointer itself (4 bytes on 32-bit, 8 bytes on 64-bit), not the size of the
buffer the pointer is pointing to.

[Noticed when someone suggested we apply the patch from Debian to our packages
 as well.]

--
    -Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#701372: [PATCH] dix: add missing include for DeleteWindowFromAnySelections

2013-07-27 Thread Alan Coopersmith

On 07/27/13 03:09 AM, Julien Cristau wrote:

Fixes build error with XACE disabled:
window.c:886:5: error: implicit declaration of function 
'DeleteWindowFromAnySelections' [-Werror=implicit-function-declaration]
  DeleteWindowFromAnySelections(pWin);
  ^

Debian bug#701372

Reported-by: Matthias Klose 
Signed-off-by: Julien Cristau 
---
  dix/window.c |1 +
  1 file changed, 1 insertion(+)

diff --git a/dix/window.c b/dix/window.c
index 8950f97..9fa51c2 100644
--- a/dix/window.c
+++ b/dix/window.c
@@ -126,6 +126,7 @@ Equipment Corporation.
  #ifdef COMPOSITE
  #include "compint.h"
  #endif
+#include "selection.h"

  #include "privates.h"
  #include "xace.h"



Reviewed-by: Alan Coopersmith 

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#488535: [PATCH:xrdb] Ensure we don't read out of ClassName array bounds for unknown visual type

2013-07-06 Thread Alan Coopersmith
Should never happen, but has been reported as happening at least once:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=488535

Instead prints warning messages to stderr to help diagnose where the
bad visual information is coming from.

(Tested by temporarily commenting out names in the ClassName array.)

Signed-off-by: Alan Coopersmith 
---
 xrdb.c |   28 ++--
 1 file changed, 22 insertions(+), 6 deletions(-)

diff --git a/xrdb.c b/xrdb.c
index b7c9fa3..d42a519 100644
--- a/xrdb.c
+++ b/xrdb.c
@@ -605,6 +605,8 @@ static char *ClassNames[] = {
 "DirectColor"
 };
 
+#define NUM_CLASS_NAMES (int)(sizeof(ClassNames) / sizeof(ClassNames[0]))
+
 static void
 DoScreenDefines(Display *display, int scrno, String *defs)
 {
@@ -625,9 +627,16 @@ DoScreenDefines(Display *display, int scrno, String *defs)
 AddNum(defs, "Y_RESOLUTION", Resolution(screen->height,screen->mheight));
 AddNum(defs, "PLANES", DisplayPlanes(display, scrno));
 AddNum(defs, "BITS_PER_RGB", visual->bits_per_rgb);
-AddDefQ(defs, "CLASS", ClassNames[visual->class]);
-snprintf(name, sizeof(name), "CLASS_%s", ClassNames[visual->class]);
-AddNum(defs, name, (int)visual->visualid);
+if (visual->class >= 0 && visual->class < NUM_CLASS_NAMES) {
+   AddDefQ(defs, "CLASS", ClassNames[visual->class]);
+   snprintf(name, sizeof(name), "CLASS_%s", ClassNames[visual->class]);
+   AddNum(defs, name, (int)visual->visualid);
+}
+else {
+   fprintf(stderr,
+   "%s: unknown visual type %d for default visual id 0x%lx\n",
+   ProgramName, visual->class, visual->visualid);
+}
 switch(visual->class) {
case StaticColor:
case PseudoColor:
@@ -643,9 +652,16 @@ DoScreenDefines(Display *display, int scrno, String *defs)
break;
}
if (j < 0) {
-   snprintf(name, sizeof(name), "CLASS_%s_%d",
-   ClassNames[vinfos[i].class], vinfos[i].depth);
-   AddNum(defs, name, (int)vinfos[i].visualid);
+   if (vinfos[i].class >= 0 && vinfos[i].class < NUM_CLASS_NAMES) {
+   snprintf(name, sizeof(name), "CLASS_%s_%d",
+ClassNames[vinfos[i].class], vinfos[i].depth);
+   AddNum(defs, name, (int)vinfos[i].visualid);
+   }
+   else {
+   fprintf(stderr,
+   "%s: unknown visual type %d for visual id 0x%lx\n",
+   ProgramName, vinfos[i].class, vinfos[i].visualid);
+   }
}
 }
 XFree((char *)vinfos);
-- 
1.7.9.2


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#621702: [PATCH libXRes] Compute number of bytes correctly in XResQueryClientPixmapBytes

2013-07-06 Thread Alan Coopersmith

On 06/30/13 11:09 AM, Julien Cristau wrote:

64-bit servers send the upper 32-bit in bytes_overflow.  We were
multiplying by 2^32 - 1 instead of 2^32 when putting things back
together.

Debian bug#621702

Reported-by: Kevin Ryde 
Signed-off-by: Julien Cristau 
---
  src/XRes.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/XRes.c b/src/XRes.c
index 51e905f..89fab82 100644
--- a/src/XRes.c
+++ b/src/XRes.c
@@ -240,7 +240,7 @@ Status XResQueryClientPixmapBytes (
  }

  #ifdef LONG64
-*bytes = (rep.bytes_overflow * 4294967295) + rep.bytes;
+*bytes = (rep.bytes_overflow * 4294967296UL) + rep.bytes;
  #else
  *bytes = rep.bytes_overflow ? 0x : rep.bytes;
  #endif



Reviewed-by: Alan Coopersmith 

--
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#657215: [PATCH videoproto] Xvproto: fix xvEncodingInfo to match actual wire protocol

2013-06-11 Thread Alan Coopersmith

On 06/ 9/13 10:00 AM, Julien Cristau wrote:

The padding is *before* the rate field, so the rate is placed on a 32bit
boundary.  This change adds explicit padding between height and rate,
and removes extraneous padding after the rate field, which the server
never sent and xlib never read.

This changes sizeof(xvEncodingInfo).  Hopefully that's not a big deal as
clients only see the Xlib structure XvEncodingInfo.

Debian bug#657215

Reported-by: Kevin Ryde 
Signed-off-by: Julien Cristau 


This looked reasonable to me, but just to be sure, I threw together the
attached test program and confirmed it doesn't change where in the
struct the rate data appears, due to the compiler silently inserting
padding to reach 32-bit alignment.

In both 32-bit & 64-bit, on both SPARC & x86, all 4 returned:

 OLD  NEW
offsetof(rate):   12  12
offsetof(pad):20  10
sizeof(struct):   24  20

which matches my expectations, so:

Reviewed-by: Alan Coopersmith 

--
    -Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc
#include 
#include 
#include 

#define XvEncodingID CARD32

typedef struct {
  INT32 numerator B32;
  INT32 denominator B32;
} xvRational;

typedef struct {
  XvEncodingID encoding B32;
  CARD16 name_size B16;
  CARD16 width B16, height B16;
  xvRational rate;
  CARD16 pad B16;
} OLDxvEncodingInfo;

typedef struct {
  XvEncodingID encoding B32;
  CARD16 name_size B16;
  CARD16 width B16, height B16;
  CARD16 pad B16;
  xvRational rate;
} NEWxvEncodingInfo;

int main(int argc, char **argv)
{
printf(" OLD  NEW\n");
printf("offsetof(rate):   %2d  %2d\n",
   offsetof(OLDxvEncodingInfo, rate),
   offsetof(NEWxvEncodingInfo, rate));
printf("offsetof(pad):%2d  %2d\n",
   offsetof(OLDxvEncodingInfo, pad),
   offsetof(NEWxvEncodingInfo, pad));
printf("sizeof(struct):   %2d  %2d\n",
   sizeof(OLDxvEncodingInfo),
   sizeof(NEWxvEncodingInfo));
}


Bug#700423: [Xcb] Bug#700423: Attaching correct diff

2013-02-12 Thread Alan Coopersmith
On 02/12/13 07:11 AM, Vibhav Pant wrote:
> The diff I had earlier attached was having some mistakes. I have attached the
> correct diff.

The previous diff was at least for libxcb - this one is for a different library,
libcaca, which seems like a bigger mistake.

(The previous one seems like it would fail if you weren't running in a session
 that can connect to an X server, which I'd expect most build environments
 cannot do.)

-- 
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#696965: [PATCH] support for HW_SKIP_CONSOLE breaks use by blind people

2012-12-31 Thread Alan Coopersmith
On 12/31/12 05:36 PM, Samuel Thibault wrote:
> Michal Suchanek, le Mon 31 Dec 2012 19:22:13 +0100, a écrit :
>> why is that patch needed?
>>
>> It is quite non-obvious why would dummy driver require a console under
>> any circumstances. It does not render anything anywhere so does not
>> use console for anything.
> 
> The console *is* needed for keyboard input.
> 
> Again, the use case is blind people who have use of possessing an actual
> screen, and thus don't have any, and thus have to use the dummy driver.

So it sounds like that should be handled by the input driver, not the
output driver.

People who want to use dummy output with real input should have input
devices configured - those who want an Xvfb like experience can use
the dummy video driver and either use the void input driver or configure
their servers to have no input drivers loaded.

-- 
-Alan Coopersmith-  alan.coopersm...@oracle.com
 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#582650: [PATCH] Xephyr.man.pre: In OPTIONS mention that Xserver(1) are also accepted

2010-06-07 Thread Alan Coopersmith
Jari Aalto wrote:
> diff --git a/hw/kdrive/ephyr/Xephyr.man.pre b/hw/kdrive/ephyr/Xephyr.man.pre
> index eb80b96..3aece2a 100644
> --- a/hw/kdrive/ephyr/Xephyr.man.pre
> +++ b/hw/kdrive/ephyr/Xephyr.man.pre
> @@ -39,6 +39,8 @@ uses the host X server window as "framebuffer" via fast SHM 
> XImages.
>  It also has support for "visually" debugging what the server is
>  painting.
>  .SH OPTIONS
> +The server accepts all the standard options of Xserver(__appmansuffix__)
> +and the following additional options:
>  .TP 8
>  .BI -screen " width" x height
>  sets the screen size.

Thanks for the patch.
Reviewed-by: Alan Coopersmith 

-- 
-Alan Coopersmith-alan.coopersm...@oracle.com
 Oracle Solaris Platform Engineering: X Window System




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#382037: Missing programname and PID in syslog-log by PAM

2008-03-04 Thread Alan Coopersmith
Julien Cristau wrote:
> It seems that xdm doesn't call openlog(), so the identifier defaults to
> null when pam calls syslog().
> Alan, do you know why the call to openlog() is conditional on
> __OpenBSD__?  I think we should add it on linux too (something like the
> attached patch).

That's the way we inherited it from XFree86 - from looking at their CVS,
this was a set of changes from the OpenBSD team that XFree86 applied for
OpenBSD only even though they were more widely applicable:

 271. xdm updates from the OpenBSD team, including:
   - support for Kerberos IV authentication [enabled only in OpenBSD.cf,
 but could be enabled for other systems by setting HasdKrbIV]
   - use the arc4random(4) random number generator on OpenBSD
   - add a new resource "allowRootLogin", which can be used to disable
 root logins through xdm
   - log failed logins to syslogd [OpenBSD only, but could be enabled on
 other systems]
   - verify that the shell is valid using /etc/shells [OpenBSD only]
   - verify that the account hasn't expired [OpenBSD only, but could be
 enabled on other system that use the same passwd aging structure]
  (#3948, 3980, OpenBSD team, Matthieu Herrb).

I'd have no objections to using openlog() on other OS'es too, though I'd
prefer an autoconf test for the openlog() function over just adding __linux__
(or just making it non-conditional for all OS'es if everyone has openlog()).

-- 
-Alan Coopersmith-   [EMAIL PROTECTED]
 Sun Microsystems, Inc. - X Window System Engineering




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]



Bug#397956: xserver-xorg-core: Xorg man page claims to be for X11R6

2007-02-01 Thread Alan Coopersmith

Upstream fixed already for xorg-server-1.2.0:

http://gitweb.freedesktop.org/?p=xorg/xserver.git;a=commitdiff;h=5f14c9a78b74017cdc66b79b63b3b1f68cf24911

--
-Alan Coopersmith-   [EMAIL PROTECTED]
 Sun Microsystems, Inc. - X Window System Engineering


--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]