MonetDB: Dec2011 - Don't assume buffers are magically null termi...

[Sjoerd Mullender]

Changeset: d69fe5a60563 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=d69fe5a60563
Modified Files:
        clients/mapiclient/stethoscope.c
Branch: Dec2011
Log Message:

Don't assume buffers are magically null terminated.


diffs (24 lines):

diff --git a/clients/mapiclient/stethoscope.c b/clients/mapiclient/stethoscope.c
--- a/clients/mapiclient/stethoscope.c
+++ b/clients/mapiclient/stethoscope.c
@@ -227,8 +227,9 @@ doProfile(void *d)
        wthread *wthr = (wthread*)d;
        int i;
        size_t a;
+       ssize_t n;
        char *response, *x;
-       char buf[BUFSIZ];
+       char buf[BUFSIZ + 1];
        char *e;
        char *mod, *fcn;
        char *host;
@@ -322,7 +323,8 @@ doProfile(void *d)
        fflush(NULL);
 
        i = 0;
-       while (mnstr_read(wthr->s, buf, 1, BUFSIZ)) {
+       while ((n = mnstr_read(wthr->s, buf, 1, BUFSIZ)) > 0) {
+               buf[n] = 0;
                response = buf;
                while ((e = strchr(response, '\n')) != NULL) {
                        *e = 0;
_______________________________________________
Checkin-list mailing list
Checkin-list@monetdb.org
http://mail.monetdb.org/mailman/listinfo/checkin-list