diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2008-03-27 15:49:09 +0000 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2008-03-27 15:49:09 +0000 |
commit | dbb50ec360556c0d4402808bf326c38831ff1734 (patch) | |
tree | bb27ecc97696b1964a1fe73c9f941a45687215a4 /debug.c | |
parent | 94e32990ee01bc7622909f40f0839bc4516cd56d (diff) | |
download | rsyslog-dbb50ec360556c0d4402808bf326c38831ff1734.tar.gz rsyslog-dbb50ec360556c0d4402808bf326c38831ff1734.tar.bz2 rsyslog-dbb50ec360556c0d4402808bf326c38831ff1734.zip |
bugfix: $ModDir did invalid bounds checking, potential overlow in
dbgprintf() - thanks to varmojfekoj for the patch
Diffstat (limited to 'debug.c')
-rw-r--r-- | debug.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -818,7 +818,7 @@ dbgoprint(obj_t *pObj, char *fmt, ...) lenWriteBuf = vsnprintf(pszWriteBuf, sizeof(pszWriteBuf), fmt, ap); if(lenWriteBuf >= sizeof(pszWriteBuf)) { /* if our buffer was too small, we simply truncate. TODO: maybe something better? */ - lenWriteBuf--; + lenWriteBuf = sizeof(pszWriteBuf) - 1; } va_end(ap); /* @@ -892,7 +892,7 @@ dbgprintf(char *fmt, ...) lenWriteBuf = vsnprintf(pszWriteBuf, sizeof(pszWriteBuf), fmt, ap); if(lenWriteBuf >= sizeof(pszWriteBuf)) { /* if our buffer was too small, we simply truncate. TODO: maybe something better? */ - lenWriteBuf--; + lenWriteBuf = sizeof(pszWriteBuf) - 1; } va_end(ap); /* |