io: for rule input don't use virtual function if class changed

root-project · Jan 19, 2025 · 4a60b1a · 4a60b1a
1 parent a07b28f
commit 4a60b1a
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/io/io/src/TStreamerInfo.cxx b/io/io/src/TStreamerInfo.cxx
@@ -288,10 +288,13 @@ namespace {
                memType = TVirtualStreamerInfo::kAnyP;
             }
          } else {
-            if (memClass->IsTObject()) {
-               memType = TVirtualStreamerInfo::kObject;
-            } else if (memClass->GetCollectionProxy()) {
+            if (memClass->GetCollectionProxy()) {
                memType = TVirtualStreamerInfo::kSTL;
+            } else if(memClass->IsTObject() && memClass == element->GetClassPointer()) {
+               // If there is a change in the class type, we can't use the TObject::Streamer
+               // virtual function: it would streame the data using the in-memory type rather
+               // than the onfile type.
+               memType = TVirtualStreamerInfo::kObject;
             } else {
                memType = TVirtualStreamerInfo::kAny;
             }
@@ -349,6 +352,13 @@ namespace {
    void UpdateFromRule(const TStreamerInfo *info, const ROOT::TSchemaRule::TSources *s, TStreamerElement *element)
    {
       auto [memClass, memType, datasize, dimensions, totaldim] = GetSourceType(s, element);
+      if (element->GetType() == TVirtualStreamerInfo::kObject && memClass != element->GetClassPointer())
+      {
+         // If there is a change in the class type, we can't use the TObject::Streamer
+         // virtual function: it would streame the data using the in-memory type rather
+         // than the onfile type.
+         element->SetType(TVirtualStreamerInfo::kAny);
+      }
       element->SetNewType( memType );
       element->SetNewClass( memClass );
       // We can not change the recorded dimensions.  Let's check that