From 7297ae129c0ea9d18945631503a6094f8d68100e Mon Sep 17 00:00:00 2001 From: Johan Wikman Date: Fri, 30 Jun 2017 08:59:03 +0200 Subject: [PATCH] MXS-1248: Pointer value alone does not tell validity The pointer pointing to a recursive select may be non-null even if it is not valid. The specific boolean must be checked as well. --- query_classifier/qc_mysqlembedded/qc_mysqlembedded.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/query_classifier/qc_mysqlembedded/qc_mysqlembedded.cc b/query_classifier/qc_mysqlembedded/qc_mysqlembedded.cc index b2ebde1f2..91a6e87ff 100644 --- a/query_classifier/qc_mysqlembedded/qc_mysqlembedded.cc +++ b/query_classifier/qc_mysqlembedded/qc_mysqlembedded.cc @@ -2909,7 +2909,7 @@ int32_t qc_mysql_get_field_info(GWBUF* buf, const QC_FIELD_INFO** infos, uint32_ { update_field_infos(pi, lex, element->spec, usage, NULL); - if (element->first_recursive) + if (element->is_recursive && element->first_recursive) { update_field_infos(pi, lex, element->first_recursive, usage, NULL); }