mirror of
https://github.com/weechat/weechat.git
synced 2026-06-26 04:46:37 +02:00
javascript: remove check of NULL pointers before calling free() (issue #865)
This commit is contained in:
@@ -1216,9 +1216,7 @@ weechat_js_api_config_option_change_cb (const void *pointer, void *data,
|
||||
WEECHAT_SCRIPT_EXEC_IGNORE,
|
||||
ptr_function,
|
||||
"ss", func_argv);
|
||||
|
||||
if (rc)
|
||||
free (rc);
|
||||
free (rc);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1243,9 +1241,7 @@ weechat_js_api_config_option_delete_cb (const void *pointer, void *data,
|
||||
WEECHAT_SCRIPT_EXEC_IGNORE,
|
||||
ptr_function,
|
||||
"ss", func_argv);
|
||||
|
||||
if (rc)
|
||||
free (rc);
|
||||
free (rc);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -2864,8 +2860,7 @@ weechat_js_api_hook_print_cb (const void *pointer, void *data,
|
||||
ret = *rc;
|
||||
free (rc);
|
||||
}
|
||||
if (func_argv[3])
|
||||
free (func_argv[3]);
|
||||
free (func_argv[3]);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
@@ -452,8 +452,7 @@ weechat_js_unload (struct t_plugin_script *script)
|
||||
{
|
||||
rc = (int *)weechat_js_exec (script, WEECHAT_SCRIPT_EXEC_INT,
|
||||
script->shutdown_func, NULL, NULL);
|
||||
if (rc)
|
||||
free (rc);
|
||||
free (rc);
|
||||
}
|
||||
|
||||
filename = strdup (script->filename);
|
||||
@@ -473,8 +472,7 @@ weechat_js_unload (struct t_plugin_script *script)
|
||||
|
||||
(void) weechat_hook_signal_send ("javascript_script_unloaded",
|
||||
WEECHAT_HOOK_SIGNAL_STRING, filename);
|
||||
if (filename)
|
||||
free (filename);
|
||||
free (filename);
|
||||
}
|
||||
|
||||
/*
|
||||
@@ -660,8 +658,7 @@ weechat_js_command_cb (const void *pointer, void *data,
|
||||
ptr_name, 1);
|
||||
weechat_js_load ((path_script) ? path_script : ptr_name,
|
||||
NULL);
|
||||
if (path_script)
|
||||
free (path_script);
|
||||
free (path_script);
|
||||
}
|
||||
else if (weechat_strcmp (argv[1], "reload") == 0)
|
||||
{
|
||||
|
||||
Reference in New Issue
Block a user