diff options
| author | Varun Gupta <varun.gupta@mariadb.com> | 2020-07-27 13:53:33 +0530 |
|---|---|---|
| committer | Varun Gupta <varun.gupta@mariadb.com> | 2020-07-27 13:53:33 +0530 |
| commit | a6410deba99d5060a8c3fc0d5f267100125dc6ac (patch) | |
| tree | 33879ee74a22882b4c612a6e31d63cbf185eb4a0 /sql/item_windowfunc.cc | |
| parent | 5f1ec5cbb78a427ff260888bef5e19daa36b10e2 (diff) | |
| download | mariadb-git-a6410deba99d5060a8c3fc0d5f267100125dc6ac.tar.gz | |
MDEV-18916: crash in Window_spec::print_partition() with decimals
Removed an unnecessary ifndef which was printing the window name for a named
window only in the case of debug build. The print() for the window function
should behave in the same way on both release and debug builds.
Diffstat (limited to 'sql/item_windowfunc.cc')
| -rw-r--r-- | sql/item_windowfunc.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sql/item_windowfunc.cc b/sql/item_windowfunc.cc index a3edacd880e..87dddbfb439 100644 --- a/sql/item_windowfunc.cc +++ b/sql/item_windowfunc.cc @@ -443,10 +443,8 @@ void Item_window_func::print(String *str, enum_query_type query_type) { window_func()->print(str, query_type); str->append(" over "); -#ifndef DBUG_OFF - if (!window_spec) // one can call dbug_print_item() anytime in gdb + if (!window_spec) str->append(window_name); else -#endif - window_spec->print(str, query_type); + window_spec->print(str, query_type); } |