From: Tobias Brunner <tobias@strongswan.org>
Date: Mon, 15 Dec 2014 17:18:40 +0000 (+0100)
Subject: pts: Do not override status with SUCCESS if length is invalid
X-Git-Tag: 5.2.2~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=027c19b09756f403e6a73849736dd481a959a507;p=thirdparty%2Fstrongswan.git

pts: Do not override status with SUCCESS if length is invalid
---

diff --git a/src/libimcv/tcg/pts/tcg_pts_attr_file_meas.c b/src/libimcv/tcg/pts/tcg_pts_attr_file_meas.c
index 5b4cc273ba..397882926a 100644
--- a/src/libimcv/tcg/pts/tcg_pts_attr_file_meas.c
+++ b/src/libimcv/tcg/pts/tcg_pts_attr_file_meas.c
@@ -242,6 +242,8 @@ METHOD(pa_tnc_attr_t, process, status_t,
 		this->count--;
 	}
 
+	status = SUCCESS;
+
 	if (this->length != this->offset)
 	{
 		DBG1(DBG_TNC, "inconsistent length for %N/%N", pen_names, PEN_TCG,
@@ -249,7 +251,6 @@ METHOD(pa_tnc_attr_t, process, status_t,
 		*offset = this->offset;
 		status = FAILED;
 	}
-	status = SUCCESS;
 
 end:
 	reader->destroy(reader);