Fixed: segv on exit

Test: rm openconfig-interface
clicon · Jan 10, 2024 · d5a8d6b · d5a8d6b
1 parent 0ec1e0a
commit d5a8d6b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 210 deletions.
diff --git a/apps/backend/backend_main.c b/apps/backend/backend_main.c
@@ -141,11 +141,11 @@ backend_terminate(clixon_handle h)
         unlink(pidfile);   
     if (sockfamily==AF_UNIX && lstat(sockpath, &st) == 0)
         unlink(sockpath);
-    backend_handle_exit(h); /* Also deletes streams. Cannot use h after this. */
     clixon_event_exit();
     clixon_debug(CLIXON_DBG_CLIENT, "%s done", __FUNCTION__);
     clixon_err_exit();
     clixon_log_exit();
+    backend_handle_exit(h); /* Also deletes streams. Cannot use h after this. */
     return 0;
 }
 

diff --git a/lib/src/clixon_datastore_write.c b/lib/src/clixon_datastore_write.c
@@ -1213,6 +1213,7 @@ text_modify_top(clixon_handle       h,
  *     cbret contains netconf error message
  * @endcode
  * @note if xret is non-null, it may contain error message
+ * @note x1 may change as a side-effect (eg operation attributes are stripped)
  */
 int
 xmldb_put(clixon_handle       h,

diff --git a/test/test_openconfig_interfaces.sh b/test/test_openconfig_interfaces.sh