From: František Dvořák Date: Tue, 27 Jan 2009 12:44:35 +0000 (+0000) Subject: Always return nonzero on error after parsing (catched from one memleak in notifications). X-Git-Tag: glite-lb-types_R_1_0_0_3~4 X-Git-Url: http://scientific.zcu.cz/git/?a=commitdiff_plain;h=d6a45847d71d6cf01610438e59b0f6f07130e8fa;p=jra1mw.git Always return nonzero on error after parsing (catched from one memleak in notifications). --- 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 */