[MDEV-30680] Warning: Memory not freed: 280 on mangled query, LeakSanitizer: detected memory leaks - Jira

Details

Type: Bug
Status: Closed (View Workflow)
Priority: Major
Resolution: Fixed
Affects Version/s: 10.5, 10.6, 10.7(EOL), 10.8(EOL), 10.9(EOL), 10.10(EOL), 10.11, 11.0(EOL), 11.1(EOL)
Fix Version/s: 10.8.8, 10.5.22, 10.6.15, 10.9.8, 10.10.6, 10.11.5, 11.0.3, 11.1.2
Component/s: Stored routines
Labels:

Description

Present in 10.5+ only, both in debug and optimized.

IF(NOT (scalar (@0s)>scalar (@0s))) { die (.);�ku u va�oj SQL sINTaksi{ die (. na liniji0;;

SHUTDOWN;

Leads to (ref error log after shutdown):

11.0.1 f2dc4d4c10ac36a73b5c1eb765352d3aee808d66 (Debug)
2023-02-18 14:22:07 0 [Note] /test/MD180223-mariadb-11.0.1-linux-x86_64-dbg/bin/mariadbd: Shutdown complete
Warning: Memory not freed: 280

In all affected versions.

No UBSAN/ASAN issues observed.

I tried to make this work under MTR for some time, but failed. AFAICS ftm, this issue can only be reproduced in a client, not in MTR, not even when using the CLI via MTR i.e. --start-and-exit.

As such, you will need to start mariadbd and then mariadb manually/directly to reproduce.

Attachments

Issue Links

relates to

MDEV-26186 280 Bytes lost in mysys/array.c, mysys/hash.c, sql/sp.cc, sql/sp.cc, sql/item_create.cc, sql/item_create.cc, sql/sql_yacc.yy:10748 when using oracle sql_mode

Closed

MDEV-31578 DECLARE CURSOR: "Memory not freed: 280 bytes lost" on syntax error

Closed

Activity

Ascending order - Click to sort in descending order

View 6 older comments

Roel Van de Paar added a comment - 2023-06-08 03:05

shulga Could you have a look at the patch please? We see this error somewhat regularly in testing. Thank you!

Roel Van de Paar added a comment - 2023-06-08 03:05 shulga Could you have a look at the patch please? We see this error somewhat regularly in testing. Thank you!

Roel Van de Paar added a comment - 2023-06-14 00:47 - edited

Please also test

IF(NOT (SCALAR (@0s)>SCALAR (@0s))) { die (.);

SHUTDOWN;

Against LSAN.

LSAN|memory leak|mysys/my_malloc.c|malloc|my_malloc|init_dynamic_array2|sp_add_used_routine

Roel Van de Paar added a comment - 2023-06-14 00:47 - edited Please also test IF ( NOT (SCALAR (@0s)>SCALAR (@0s))) { die (.); SHUTDOWN; Against LSAN. LSAN|memory leak|mysys/my_malloc.c|malloc|my_malloc|init_dynamic_array2|sp_add_used_routine

Dmitry Shulga added a comment - 2023-06-20 07:48

The patch approved

Dmitry Shulga added a comment - 2023-06-20 07:48 The patch approved

Alexander Barkov added a comment - 2023-06-22 03:03 - edited

The problem is also repeatable with:

DELIMITER $$

BEGIN NOT ATOMIC

  DECLARE cur CURSOR (a INT) FOR SELECT a+1;

  OPEN cur(sp_followed_by_syntax_error();

  CLOSE cur;

END;

$$

DELIMITER ;

It's now reported as a separate issue ~~MDEV-31578~~.

Alexander Barkov added a comment - 2023-06-22 03:03 - edited The problem is also repeatable with: DELIMITER $$ BEGIN NOT ATOMIC DECLARE cur CURSOR (a INT ) FOR SELECT a+1; OPEN cur(sp_followed_by_syntax_error(); CLOSE cur; END ; $$ DELIMITER ; It's now reported as a separate issue MDEV-31578 .

Alexander Barkov added a comment - 2023-06-22 12:03

The new patch is here:

https://github.com/MariaDB/server/commit/575ab6d3fd8d2d311d55391d54f3f665c3ae07ac

Alexander Barkov added a comment - 2023-06-22 12:03 The new patch is here: https://github.com/MariaDB/server/commit/575ab6d3fd8d2d311d55391d54f3f665c3ae07ac

People

Assignee:: Alexander Barkov

Reporter:: Roel Van de Paar

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 2023-02-18 03:20

Updated:: 2023-07-18 02:26

Resolved:: 2023-06-29 10:48

Git Integration

Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.

MariaDB Server