From 33fdc805b1254934f496c6e2d34856626da02aa7 Mon Sep 17 00:00:00 2001 From: Andreas Gerstmayr Date: Mon, 12 Jul 2021 14:12:17 +0200 Subject: [PATCH] fix possible null dereference Make sure that pLimitN is not `nil` before dereferencing the pointer. --- internal/statements/query.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/statements/query.go b/internal/statements/query.go index a972a8e0..69f48e73 100644 --- a/internal/statements/query.go +++ b/internal/statements/query.go @@ -314,7 +314,7 @@ func (statement *Statement) genSelectSQL(columnStr string, needLimit, needOrderB fmt.Fprint(&buf, " LIMIT ", *pLimitN) } } else if dialect.URI().DBType == schemas.ORACLE { - if statement.Start != 0 || pLimitN != nil { + if statement.Start != 0 && pLimitN != nil { oldString := buf.String() buf.Reset() rawColStr := columnStr -- 2.40.1