diff options
| author | Quentin Rameau <quinq@fifth.space> | 2018-10-09 12:27:59 +0200 |
|---|---|---|
| committer | Quentin Rameau <quinq@fifth.space> | 2018-10-15 13:15:54 +0200 |
| commit | 02541c314678aa1ee6eb485ca9dce66a46693aac (patch) | |
| tree | 3938c54325e4defb38ed9dfb296dd5651c02cd83 /surf.c | |
| parent | Use pkg-config for X11{INC,LIB} (diff) | |
| download | surf-02541c314678aa1ee6eb485ca9dce66a46693aac.tar.gz surf-02541c314678aa1ee6eb485ca9dce66a46693aac.tar.bz2 surf-02541c314678aa1ee6eb485ca9dce66a46693aac.zip | |
Send message size inside messages through pipes
Diffstat (limited to 'surf.c')
| -rw-r--r-- | surf.c | 26 |
1 files changed, 17 insertions, 9 deletions
diff --git a/surf.c b/surf.c index d48fbc9..2b54e3c 100644 --- a/surf.c +++ b/surf.c @@ -1209,20 +1209,22 @@ newview(Client *c, WebKitWebView *rv) static gboolean readpipe(GIOChannel *s, GIOCondition ioc, gpointer unused) { - char msg[MSGBUFSZ]; - gsize msgsz; + static char msg[MSGBUFSZ], msgsz; GError *gerr = NULL; - if (g_io_channel_read_chars(s, msg, sizeof(msg), &msgsz, &gerr) != + if (g_io_channel_read_chars(s, msg, sizeof(msg), NULL, &gerr) != G_IO_STATUS_NORMAL) { fprintf(stderr, "surf: error reading pipe: %s\n", gerr->message); g_error_free(gerr); return TRUE; } - msg[msgsz] = '\0'; + if ((msgsz = msg[0]) < 3) { + fprintf(stderr, "surf: message too short: %d\n", msgsz); + return TRUE; + } - switch (msg[1]) { + switch (msg[2]) { case 'i': close(pipein[1]); close(pipeout[0]); @@ -1843,12 +1845,18 @@ zoom(Client *c, const Arg *a) static void msgext(Client *c, char type, const Arg *a) { - char msg[MSGBUFSZ] = { c->pageid, type, a->i, '\0' }; + static char msg[MSGBUFSZ]; + int ret; - if (pipeout[1]) { - if (write(pipeout[1], msg, sizeof(msg)) < 0) - fprintf(stderr, "surf: error sending: %s\n", msg); + if ((ret = snprintf(msg, sizeof(msg), "%c%c%c%c", + 4, c->pageid, type, a->i)) + >= sizeof(msg)) { + fprintf(stderr, "surf: message too long: %d\n", ret); + return; } + + if (pipeout[1] && write(pipeout[1], msg, sizeof(msg)) < 0) + fprintf(stderr, "surf: error sending: %.*s\n", ret-2, msg+2); } void |