summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRainer Gerhards <rgerhards@adiscon.com>2008-04-04 12:05:28 +0000
committerRainer Gerhards <rgerhards@adiscon.com>2008-04-04 12:05:28 +0000
commit91e02eb209a6571429a00fa71854709dfc18ac99 (patch)
tree553f7e49a37a359d7a5abf075ad466e4cab24c70
parent798acf51b482387a2702bd4064d538d7b3bedaa3 (diff)
downloadrsyslog-91e02eb209a6571429a00fa71854709dfc18ac99.tar.gz
rsyslog-91e02eb209a6571429a00fa71854709dfc18ac99.tar.xz
rsyslog-91e02eb209a6571429a00fa71854709dfc18ac99.zip
bugfix: zero-length strings were not supported in object deserializer
-rw-r--r--ChangeLog2
-rw-r--r--obj.c4
2 files changed, 4 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index 5926cc3f..7b18ef73 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,8 @@ Version 3.15.1 (rgerhards), 2008-04-??
- disabled atomic operations for the time being because they introduce some
cross-platform trouble - need to see how to fix this in the best
possible way
+- bugfix: zero-length strings were not supported in object
+ deserializer
- added librelp check via PKG_CHECK thanks to Michael Biebl's patch
- file relputil.c deleted, is not actually needed
- added more meaningful error messages to rsyslogd (when some errors
diff --git a/obj.c b/obj.c
index d408c4bb..2f16669a 100644
--- a/obj.c
+++ b/obj.c
@@ -495,7 +495,7 @@ finalize_it:
}
-/* de-serialize a string, length must be provided */
+/* de-serialize a string, length must be provided but may be 0 */
static rsRetVal objDeserializeStr(cstr_t **ppCStr, int iLen, strm_t *pStrm)
{
DEFiRet;
@@ -504,7 +504,7 @@ static rsRetVal objDeserializeStr(cstr_t **ppCStr, int iLen, strm_t *pStrm)
cstr_t *pCStr = NULL;
assert(ppCStr != NULL);
- assert(iLen > 0);
+ assert(iLen >= 0);
CHKiRet(rsCStrConstruct(&pCStr));