From d6a45847d71d6cf01610438e59b0f6f07130e8fa Mon Sep 17 00:00:00 2001 From: =?utf8?q?Franti=C5=A1ek=20Dvo=C5=99=C3=A1k?= Date: Tue, 27 Jan 2009 12:44:35 +0000 Subject: [PATCH] Always return nonzero on error after parsing (catched from one memleak in notifications). --- org.glite.lb.common/src/xml_parse.c.T | 27 ++++++++++++++++++--------- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/org.glite.lb.common/src/xml_parse.c.T b/org.glite.lb.common/src/xml_parse.c.T index 7c8e674..7bdcfe0 100644 --- a/org.glite.lb.common/src/xml_parse.c.T +++ b/org.glite.lb.common/src/xml_parse.c.T @@ -1418,7 +1418,8 @@ edg_wll_ErrorCode edg_wll_ParseQueryJobs(edg_wll_Context ctx, char *messageBody, if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } @@ -1504,7 +1505,8 @@ edg_wll_ErrorCode edg_wll_ParseQueryEvents(edg_wll_Context ctx, char *messageBod if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2003,7 +2005,8 @@ edg_wll_ErrorCode edg_wll_ParsePurgeResult(edg_wll_Context ctx, char *messageBod if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2064,7 +2067,8 @@ edg_wll_ErrorCode edg_wll_ParseDumpResult(edg_wll_Context ctx, char *messageBody if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2125,7 +2129,8 @@ edg_wll_ErrorCode edg_wll_ParseLoadResult(edg_wll_Context ctx, char *messageBody if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2198,7 +2203,8 @@ edg_wll_ErrorCode edg_wll_ParseIndexedAttrs(edg_wll_Context ctx, char *messageBo if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2258,7 +2264,8 @@ edg_wll_ErrorCode edg_wll_ParseNotifResult(edg_wll_Context ctx, char *messageBod if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2319,7 +2326,8 @@ edg_wll_ErrorCode edg_wll_ParseQuerySequenceCodeResult(edg_wll_Context ctx, char if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ @@ -2389,7 +2397,8 @@ edg_wll_ErrorCode edg_wll_ParseStatsResult(edg_wll_Context ctx, char *messageBod if (XMLCtx.errDesc || XMLCtx.errCode) { ctx->errDesc = XMLCtx.errDesc; - ctx->errCode = XMLCtx.errCode; + ctx->errCode = XMLCtx.errCode; + errorCode = XMLCtx.errCode; } /* print all warning if corresponding env variable is set */ -- 1.8.2.3