Commit 1c9c4423 authored by Pierre Ynard's avatar Pierre Ynard Committed by Jean-Baptiste Kempf

lua: fix more sd input options and lua stack use-after-gc

Strings from the lua stack stop being valid after you pop them
(cherry picked from commit 8f048e9cabebeec8b514e981993d16acfbbd7c28)
Signed-off-by: default avatarJean-Baptiste Kempf <jb@videolan.org>
parent a27a39a7
......@@ -224,19 +224,23 @@ static int vlclua_sd_add_item( lua_State *L )
lua_getfield( L, -1, "path" );
if( lua_isstring( L, -1 ) )
{
char **ppsz_options = NULL;
int i_options = 0;
const char *psz_path = lua_tostring( L, -1 );
lua_pop( L, 1 );
vlclua_read_options( p_sd, L, &i_options, &ppsz_options );
lua_getfield( L, -1, "title" );
lua_getfield( L, -2, "title" );
const char *psz_title = luaL_checkstring( L, -1 ) ? luaL_checkstring( L, -1 ) : psz_path;
/* The table must be at the top of the stack when calling
* vlclua_read_options() */
char **ppsz_options = NULL;
int i_options = 0;
lua_pushvalue( L, -3 );
vlclua_read_options( p_sd, L, &i_options, &ppsz_options );
input_item_t *p_input = input_item_NewExt( psz_path, psz_title,
i_options,
(const char **)ppsz_options,
VLC_INPUT_OPTION_TRUSTED, -1 );
lua_pop( L, 1 );
lua_pop( L, 3 );
if( p_input )
{
......@@ -307,16 +311,21 @@ static int vlclua_node_add_subitem( lua_State *L )
lua_getfield( L, -1, "path" );
if( lua_isstring( L, -1 ) )
{
const char *psz_path = lua_tostring( L, -1 );
/* The table must be at the top of the stack when calling
* vlclua_read_options() */
char **ppsz_options = NULL;
int i_options = 0;
const char *psz_path = lua_tostring( L, -1 );
lua_pushvalue( L, -2 );
vlclua_read_options( p_sd, L, &i_options, &ppsz_options );
input_item_node_t *p_input_node = input_item_node_Create( *pp_node );
input_item_t *p_input = input_item_NewExt( psz_path,
psz_path, i_options,
(const char **)ppsz_options,
VLC_INPUT_OPTION_TRUSTED, -1 );
lua_pop( L, 1 );
lua_pop( L, 2 );
if( p_input )
{
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment