From: Greg Hudson <ghudson@mit.edu>
Date: Tue, 17 Mar 2009 21:54:51 +0000 (+0000)
Subject: Fix error handling issue in ASN.1 decoder
X-Git-Tag: krb5-1.8-alpha1~584
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=ffafb8455cde8a0c089512444457a74357a644c3;p=krb5.git

Fix error handling issue in ASN.1 decoder

In asn1_k_decode.c, check the return value of
end_sequence_of_no_tagvars_helper.

ticket: 6427
tags: pullup
target_version: 1.7

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@22107 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/lib/krb5/asn.1/asn1_k_decode.c b/src/lib/krb5/asn.1/asn1_k_decode.c
index c5d200628..25cc4cc62 100644
--- a/src/lib/krb5/asn.1/asn1_k_decode.c
+++ b/src/lib/krb5/asn.1/asn1_k_decode.c
@@ -364,7 +364,8 @@ end_sequence_of_no_tagvars_helper(asn1buf *buf, asn1buf *seqbufp,
     return retval;
 }
 #define end_sequence_of_no_tagvars(buf) \
-    end_sequence_of_no_tagvars_helper(buf, &seqbuf, seqofindef)
+    retval = end_sequence_of_no_tagvars_helper(buf, &seqbuf, seqofindef); \
+    if (retval) clean_return(retval)
 
 /*
  * Function body for a pointer decoder, which allocates a pointer