SQL Server存储过程执行成功但无返回数据，求助排查

存储过程代码 #

SET ANSI_NULLS ON
GO

SET QUOTED_IDENTIFIER ON
GO
ALTER   PROCEDURE [dbo].[errors_master](
    @search varchar(100) = null,
    @orderColumn varchar(50) = null,
    @dir int = null,
    @columnsPassed varchar(max) = null,
    @param varchar(100) = null,
    @group int = 0
)              
AS              
BEGIN

    DECLARE @whereClause NVARCHAR(MAX);
    SET @whereClause = 
    CASE
        WHEN @search != null  
            THEN 'and template like ''%' + @search + '%''  or message like ''%' + @search + '%'
        ELSE NULL
    END

    DECLARE @whereClause2 NVARCHAR(MAX);
    SET @whereClause2 = 
    CASE
        WHEN @param != null  
            THEN 'and errorid =' + @param
        ELSE NULL
    END

    DECLARE @orderByStatement NVARCHAR(MAX);
    IF (isNull(@orderColumn, '') <> '')
    BEGIN
        SET @orderByStatement = ' order by ' + @orderColumn + ' '
    END
    ELSE
    BEGIN
        SET @orderByStatement = ' order by max(etime) '
    END

    DECLARE @orderType VARCHAR(5);
    IF (isNull(@dir, 1) <> 0)
    BEGIN
        SET @orderType = ' asc '
    END
    ELSE
    BEGIN
        SET @orderType = ' desc '
    END

    DECLARE @groupBy VARCHAR(50);
    SET @groupBy = 
        CASE 
            WHEN @group = 1 THEN ' group by errorid,template,message, type, line' 
        ELSE NULL 
        END

    DECLARE @Sql NVARCHAR(MAX);
    SET @Sql =  N' select ' + @columnsPassed + ' from dbo.errors where 1=1 ' + @whereClause + ' ' + @whereClause2 + ' '
                + @groupBy + ' ' + @orderByStatement + ' ' + @orderType

    EXECUTE sp_executesql @Sql

END

GRANT EXECUTE ON [dbo].[errors_master] TO [x] AS [dbo]
GRANT EXECUTE ON [dbo].[errors_master] TO [xa] AS [dbo]
GO

问题描述 #

执行exec dbo.errors_master时，系统提示命令成功完成，但未返回任何查询结果。

问题根源及解决办法 #

1. 核心原因：@columnsPassed参数未指定导致动态SQL无效 #

存储过程中@columnsPassed默认值为null，当直接执行存储过程不传入该参数时，SQL Server中字符串与null拼接的结果为null，因此N' select ' + @columnsPassed会变成null，最终整个@Sql变量为null。执行sp_executesql null不会执行任何有效的查询语句，自然不会返回结果。

解决办法：

• 执行存储过程时显式指定@columnsPassed参数，例如：

  exec dbo.errors_master @columnsPassed = 'errorid, template, message, etime'
  

• 或者修改存储过程，给@columnsPassed设置默认列集合，比如：

  @columnsPassed varchar(max) = 'errorid, template, message, type, line, etime'
  

2. 空值判断逻辑错误（潜在问题） #

存储过程中使用@search != null和@param != null判断空值是错误的：SQL中null不等于任何值（包括自身），!= null的结果永远是unknown，导致对应的CASE分支永远不会触发。正确的空值判断应该用IS NOT NULL，同时需要给OR条件添加括号避免逻辑冲突：

SET @whereClause = 
CASE
    WHEN @search IS NOT NULL  
        THEN 'and (template like ''%' + @search + '%''  or message like ''%' + @search + '%'')'
    ELSE NULL
END

3. 未分组时聚合函数排序的逻辑问题 #

当@group=0时，@groupBy为null，此时ORDER BY max(etime)是对整个表的etime取最大值来排序，逻辑上没有意义，且在严格SQL模式下可能触发错误。建议修改为直接按etime排序：

ELSE
BEGIN
    SET @orderByStatement = ' order by etime '
END

4. SQL注入风险（额外优化） #

当前动态SQL直接拼接用户输入参数，存在严重SQL注入风险。建议改用参数化查询，示例如下：

-- 修改where子句的拼接逻辑
DECLARE @whereClause NVARCHAR(MAX);
SET @whereClause = 
CASE
    WHEN @search IS NOT NULL  
        THEN 'and (template like ''%'' + @searchParam + ''%'' or message like ''%'' + @searchParam + ''%'')'
    ELSE ''
END

DECLARE @whereClause2 NVARCHAR(MAX);
SET @whereClause2 = 
CASE
    WHEN @param IS NOT NULL  
        THEN 'and errorid = @paramVal'
    ELSE ''
END

-- 执行时传入参数
EXECUTE sp_executesql @Sql, N'@searchParam varchar(100), @paramVal varchar(100)', 
    @searchParam = @search, @paramVal = @param

内容的提问来源于stack exchange，提问作者Rodney