Out-of-bounds read exists in the function modifySoname

yairKoskas · yairKoskas · commit e9d339465963 · 2022-12-27T09:19:52.000+02:00
diff --git a/src/patchelf.cc b/src/patchelf.cc
@@ -1272,6 +1272,7 @@ void ElfFile<ElfFileParamNames>::modifySoname(sonameMode op, const std::string &
         if (rdi(dyn->d_tag) == DT_SONAME) {
             dynSoname = dyn;
             soname = strTab + rdi(dyn->d_un.d_val);
+            checkPointer(fileContents, strTab, rdi(dyn->d_un.d_val));
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1272,6 +1272,7 @@ void ElfFile<ElfFileParamNames>::modifySoname(sonameMode op, const std::string &`
`1272`	`1272`	`if (rdi(dyn->d_tag) == DT_SONAME) {`
`1273`	`1273`	`dynSoname = dyn;`
`1274`	`1274`	`soname = strTab + rdi(dyn->d_un.d_val);`
	`1275`	`+ checkPointer(fileContents, strTab, rdi(dyn->d_un.d_val));`
`1275`	`1276`	`}`
`1276`	`1277`	`}`
`1277`	`1278`