Double-free Cocoa fix: Redux

Summary: Need get_true_type to evaluate string status

Reviewed By: mcslee

Other Notes: Submission from Andrew McGeachie

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665362 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/compiler/cpp/src/generate/t_cocoa_generator.cc b/compiler/cpp/src/generate/t_cocoa_generator.cc
index 9ade7fd..fc85365 100644 (file)
--- a/compiler/cpp/src/generate/t_cocoa_generator.cc
+++ b/compiler/cpp/src/generate/t_cocoa_generator.cc
@@ -486,7 +486,7 @@ void t_cocoa_generator::generate_cocoa_struct_reader(ofstream& out,
         // is now retaining it
         if (type_can_be_null((*f_iter)->get_type())) {
           // deserialized strings are autorelease, so don't release them
-          if (!((*f_iter)->get_type()->is_string())) {
+          if (!(get_true_type((*f_iter)->get_type())->is_string())) {
             indent(out) << "[fieldValue release];" << endl;
           }
         }