#include <stdio.h>
#include <stdlib.h>
+#include <time.h>
#include <sys/time.h>
#include <sys/types.h>
if (parent)
abort ();
if (! (kids && nkids))
- abort ();
+ return 0;
for (i = 0; i < nkids; i++)
{
Atom type;
int format;
unsigned long nitems, bytesafter;
- char *v;
+ unsigned char *v;
int status;
/* We're walking the list of root-level windows and trying to find
XA_SCREENSAVER_VERSION,
0, 200, False, XA_STRING,
&type, &format, &nitems, &bytesafter,
- (unsigned char **) &v);
+ &v);
XSync (dpy, False);
XSetErrorHandler (old_handler);
old_handler = 0;
if (status == Success && type != None)
{
+ Window ret = kids[i];
if (version)
- *version = v;
- return kids[i];
+ *version = (char *) v;
+ XFree (kids);
+ return ret;
}
}
+
+ if (kids) XFree (kids);
return 0;
}
static int
send_xscreensaver_command (Display *dpy, Atom command, long arg,
- Window *window_ret)
+ Window *window_ret, char **error_ret)
{
+ int status = -1;
char *v = 0;
Window window = find_screensaver_window (dpy, &v);
XWindowAttributes xgwa;
+ char err[2048];
if (window_ret)
*window_ret = window;
if (!window)
{
+ sprintf (err, "no screensaver is running on display %s",
+ DisplayString (dpy));
+
+ if (error_ret)
+ {
+ *error_ret = strdup (err);
+ status = -1;
+ goto DONE;
+ }
+
if (command == XA_EXIT)
- return 1;
+ {
+ /* Don't print an error if xscreensaver is already dead. */
+ status = 1;
+ goto DONE;
+ }
- /* Don't print this if xscreensaver is already dead. */
- fprintf (stderr, "%s: no screensaver is running on display %s\n",
- progname, DisplayString (dpy));
- return -1;
+ fprintf (stderr, "%s: %s\n", progname, err);
+ status = -1;
+ goto DONE;
}
/* Select for property change events, so that we can read the response. */
memset (&hint, 0, sizeof(hint));
if (!v || !*v)
{
- fprintf (stderr, "%s: version property not set on window 0x%x?\n",
- progname, (unsigned int) window);
- return -1;
+ sprintf (err, "version property not set on window 0x%x?",
+ (unsigned int) window);
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+
+ status = -1;
+ goto DONE;
}
XGetClassHint(dpy, window, &hint);
if (!hint.res_class)
{
- fprintf (stderr, "%s: class hints not set on window 0x%x?\n",
- progname, (unsigned int) window);
- return -1;
+ sprintf (err, "class hints not set on window 0x%x?",
+ (unsigned int) window);
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+
+ if (v) free (v);
+ status = -1;
+ goto DONE;
}
fprintf (stdout, "%s %s", hint.res_class, v);
Atom type;
int format;
unsigned long nitems, bytesafter;
- CARD32 *data = 0;
+ unsigned char *dataP = 0;
if (XGetWindowProperty (dpy,
RootWindow (dpy, 0),
XA_SCREENSAVER_STATUS,
0, 999, False, XA_INTEGER,
&type, &format, &nitems, &bytesafter,
- (unsigned char **) &data)
+ &dataP)
== Success
&& type
- && data)
+ && dataP)
{
Atom blanked;
time_t tt;
char *s;
+ Atom *data = (Atom *) dataP;
if (type != XA_INTEGER || nitems < 3)
{
if (data) free (data);
fprintf (stdout, "\n");
fflush (stdout);
- fprintf (stderr, "%s: bad status format on root window.\n",
- progname);
- return -1;
+ fprintf (stderr, "bad status format on root window.\n");
+ status = -1;
+ goto DONE;
}
blanked = (Atom) data[0];
}
if (any && nhacks == 1)
- fprintf (stdout, " (hack #%d)\n", data[2]);
+ fprintf (stdout, " (hack #%d)\n", (int) data[2]);
else if (any)
{
fprintf (stdout, " (hacks: ");
for (i = 0; i < nhacks; i++)
{
- fprintf (stdout, "#%d", data[2 + i]);
+ fprintf (stdout, "#%d", (int) data[2 + i]);
if (i != nhacks-1)
fputs (", ", stdout);
}
}
else
{
- if (data) free (data);
+ if (dataP) XFree (dataP);
fprintf (stdout, "\n");
fflush (stdout);
- fprintf (stderr, "%s: no saver status on root window.\n",
- progname);
- return -1;
+ fprintf (stderr, "no saver status on root window.\n");
+ status = -1;
+ goto DONE;
}
}
/* No need to read a response for these commands. */
- return 1;
+ status = 1;
+ goto DONE;
}
else
{
event.xclient.data.l[2] = arg2;
if (! XSendEvent (dpy, window, False, 0L, &event))
{
- fprintf (stderr, "%s: XSendEvent(dpy, 0x%x ...) failed.\n",
- progname, (unsigned int) window);
- return -1;
+ sprintf (err, "XSendEvent(dpy, 0x%x ...) failed.\n",
+ (unsigned int) window);
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+ status = -1;
+ goto DONE;
}
}
+
+ status = 0;
+
+ DONE:
+ if (v) free (v);
XSync (dpy, 0);
- return 0;
+ return status;
}
static int
xscreensaver_command_response (Display *dpy, Window window,
- Bool verbose_p, Bool exiting_p)
+ Bool verbose_p, Bool exiting_p,
+ char **error_ret)
{
int fd = ConnectionNumber (dpy);
int timeout = 10;
int status;
fd_set fds;
struct timeval tv;
+ char err[2048];
while (1)
{
if (status < 0)
{
char buf[1024];
- sprintf (buf, "%s: waiting for reply", progname);
- perror (buf);
+ if (error_ret)
+ {
+ sprintf (buf, "error waiting for reply");
+ *error_ret = strdup (buf);
+ }
+ else
+ {
+ sprintf (buf, "%s: error waiting for reply", progname);
+ perror (buf);
+ }
return status;
}
else if (status == 0)
{
- fprintf (stderr, "%s: no response to command.\n", progname);
+ sprintf (err, "no response to command.");
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
return -1;
}
else
Atom type;
int format;
unsigned long nitems, bytesafter;
- char *msg = 0;
+ unsigned char *msg = 0;
XSync (dpy, False);
if (old_handler) abort();
0, 1024, True,
AnyPropertyType,
&type, &format, &nitems, &bytesafter,
- (unsigned char **) &msg);
+ &msg);
XSync (dpy, False);
XSetErrorHandler (old_handler);
old_handler = 0;
if (exiting_p)
return 0;
- fprintf (stderr,
- "%s: xscreensaver window unexpectedly deleted.\n",
- progname);
+ sprintf (err, "xscreensaver window unexpectedly deleted.");
+
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+
return -1;
}
{
if (type != XA_STRING || format != 8)
{
- fprintf (stderr,
- "%s: unrecognized response property.\n",
- progname);
+ sprintf (err, "unrecognized response property.");
+
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+
if (msg) XFree (msg);
return -1;
}
else if (!msg || (msg[0] != '+' && msg[0] != '-'))
{
- fprintf (stderr,
- "%s: unrecognized response message.\n",
- progname);
+ sprintf (err, "unrecognized response message.");
+
+ if (error_ret)
+ *error_ret = strdup (err);
+ else
+ fprintf (stderr, "%s: %s\n", progname, err);
+
if (msg) XFree (msg);
return -1;
}
else
{
int ret = (msg[0] == '+' ? 0 : -1);
- if (verbose_p || ret != 0)
- fprintf ((ret < 0 ? stderr : stdout),
- "%s: %s\n",
- progname,
- msg+1);
+ sprintf (err, "%s: %s\n", progname, (char *) msg+1);
+
+ if (error_ret)
+ *error_ret = strdup (err);
+ else if (verbose_p || ret != 0)
+ fprintf ((ret < 0 ? stderr : stdout), "%s\n", err);
+
XFree (msg);
return ret;
}
int
-xscreensaver_command (Display *dpy, Atom command, long arg, Bool verbose_p)
+xscreensaver_command (Display *dpy, Atom command, long arg, Bool verbose_p,
+ char **error_ret)
{
Window w = 0;
- int status = send_xscreensaver_command (dpy, command, arg, &w);
+ int status = send_xscreensaver_command (dpy, command, arg, &w, error_ret);
if (status == 0)
status = xscreensaver_command_response (dpy, w, verbose_p,
- (command == XA_EXIT));
+ (command == XA_EXIT),
+ error_ret);
fflush (stdout);
fflush (stderr);
if (version_ret)
{
- char *v = 0;
+ unsigned char *v = 0;
XGetWindowProperty (dpy, window, XA_SCREENSAVER_VERSION, 0, 1,
False, XA_STRING, &type, &format, &nitems,
- &bytesafter, (unsigned char **) &v);
+ &bytesafter, &v);
if (v)
{
- *version_ret = strdup (v);
+ *version_ret = strdup ((char *) v);
XFree (v);
}
}
if (user_ret || host_ret)
{
- char *id = 0;
+ unsigned char *id = 0;
const char *user = 0;
const char *host = 0;
XGetWindowProperty (dpy, window, XA_SCREENSAVER_ID, 0, 512,
False, XA_STRING, &type, &format, &nitems,
- &bytesafter, (unsigned char **) &id);
+ &bytesafter, &id);
if (id && *id)
{
const char *old_tag = " on host ";
- const char *s = strstr (id, old_tag);
+ const char *s = strstr ((char *) id, old_tag);
if (s)
{
/* found ID of the form "1234 on host xyz". */
else
{
char *o = 0, *p = 0, *c = 0;
- o = strchr (id, '(');
+ o = strchr ((char *) id, '(');
if (o) p = strchr (o, '@');
if (p) c = strchr (p, ')');
if (c)