Set interpreter only when necessary

If the given interpreter is already set, nothing needs to be done. As with modifySoname(), it skips unnecessary processing.
NixOS · Nov 18, 2024 · 0a64a88 · 0a64a88
1 parent fd9c6de
commit 0a64a88
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/patchelf.cc b/src/patchelf.cc
@@ -1459,6 +1459,11 @@ void ElfFile<ElfFileParamNames>::modifySoname(sonameMode op, const std::string &
 template<ElfFileParams>
 void ElfFile<ElfFileParamNames>::setInterpreter(const std::string & newInterpreter)
 {
+    if (getInterpreter() == newInterpreter) {
+        debug("given interpreter is already set\n");
+        return;
+    }
+
     std::string & section = replaceSection(".interp", newInterpreter.size() + 1);
     setSubstr(section, 0, newInterpreter + '\0');
     changed = true;