commit aa82e74b08fcdc2e4e6c1f6796699566b331b656
author Stephen Hines <srhines@google.com> Wed Apr 11 12:38:17 2012 -0700
committer Stephen Hines <srhines@google.com> Wed Apr 11 12:38:17 2012 -0700
tree dd6afc026135892f708fcb234d6453c3ef61dde6
parent d5a84f6d49d64738e4bb7c9dea7242e48acad959

Multiple levels of pointers cannot be exported.

Change-Id: I533d26827fa650a481df65b2e20ac6dbc3559244

slang_rs_export_type.cpp

diff --git a/slang_rs_export_type.cpp b/slang_rs_export_type.cpp
index 4ab4af2..48d2288 100644
--- a/slang_rs_export_type.cpp
+++ b/slang_rs_export_type.cpp
@@ -260,8 +260,8 @@
     }
     case clang::Type::Pointer: {
       if (TopLevelRecord) {
-        ReportTypeError(DiagEngine, NULL, TopLevelRecord,
-          "structures containing pointers cannot be exported: '%0'");
+        ReportTypeError(DiagEngine, VD, TopLevelRecord,
+            "structures containing pointers cannot be exported: '%0'");
         return NULL;
       }
 
@@ -269,8 +269,11 @@
         UNSAFE_CAST_TYPE(const clang::PointerType, T);
       const clang::Type *PointeeType = GET_POINTEE_TYPE(PT);
 
-      if (PointeeType->getTypeClass() == clang::Type::Pointer)
-        return T;
+      if (PointeeType->getTypeClass() == clang::Type::Pointer) {
+        ReportTypeError(DiagEngine, VD, TopLevelRecord,
+            "multiple levels of pointers cannot be exported: '%0'");
+        return NULL;
+      }
       // We don't support pointer with array-type pointee or unsupported pointee
       // type
       if (PointeeType->isArrayType() ||