Commit Diff

Commit:
96706b7b7e860f23ee59913e347f9b5fd306b662
From:
Ilya Verbin <iverbin@tarantool.org>
Via:
Vladimir Davydov <vdavydov.dev@gmail.com>
Date:
Message:
box: fix memory leak in lbox_key_def_new and luaT_key_def_merge There was a typo in the commit 55295f5f1be7 ("box: populate index_object.parts with key_def module methods"): the function luaT_push_key_def() was erroneously used instead of luaT_push_key_def_nodup(). Follow-up #7356 NO_DOC=bug fix NO_TEST=memory leak NO_CHANGELOG=unreleased
Actions:
Patch | Tree
commit - 62d03f15aa80abb325336f4e37b6b2ffe2917612
commit + 96706b7b7e860f23ee59913e347f9b5fd306b662
blob - b520251ea1ea0d2ee1936a76c75f8d2b9f2b5d98
blob + 0c8ee41a7055cb66f0522e057eb3d81acc345670
--- src/box/lua/key_def.c
+++ src/box/lua/key_def.c
@@ -397,7 +397,7 @@ luaT_key_def_merge(struct lua_State *L, int idx_a, int
 	if (new_key_def == NULL)
 		return luaT_error(L);
 
-	luaT_push_key_def(L, new_key_def);
+	luaT_push_key_def_nodup(L, new_key_def);
 	return 1;
 }
 
@@ -522,7 +522,7 @@ lbox_key_def_new(struct lua_State *L)
 	 */
 	key_def_update_optionality(key_def, 0);
 
-	luaT_push_key_def(L, key_def);
+	luaT_push_key_def_nodup(L, key_def);
 	return 1;
 }