直接用户输入的动态 LINQ 有什么危险吗？

Question

我在 ASP.NET MVC 应用程序中有一个表，我希望可以使用 AJAX 对其进行排序（服务器端）和过滤。 我希望它在其他地方相当容易使用，并且不想将排序和过滤硬编码到查询表达式中，因此我寻找一种动态构建表达式的方法，我发现最好的方法是使用动态 LINQ 。

来自如下 URL 的用户输入将直接插入到动态Where 或OrderBy 中。

/Orders?sortby=OrderID&order=desc&CustomerName=Microsoft

这将导致两个表达式：

OrderBy("OrderID descending")
Where(@"CustomerName.Contains(""Microsoft"")")

虽然我知道它不会直接扔到数据库中，并且在这里直接插入 SQL 是行不通的，因为它无法反映到属性并且它是类型安全的，但我不知道是否有比我更有创造力的人能够找到一种方法来利用它。 我能想到的一个漏洞是，可以对表中不可见的属性进行排序/过滤，但这并没有那么有害，因为它们仍然不会显示，并且可以通过散列来防止。

我允许直接用户输入的唯一方法是使用 OrderBy 和Where。

只是确认一下，谢谢:)

Answer 1

由于 LINQ to SQL 使用类型安全的数据模型类，因此默认情况下您可以免受 SQL 注入攻击。 LINQ to SQL 将根据基础数据类型自动对值进行编码。
(c)斯科特·古

但是你仍然可以在那里得到“除以零”，所以建议处理所有意外的异常并限制有效条目的长度，JIC

Answer 2

嗯...我刚刚发现 Dynamic Linq 至少存在一个可能的问题。 只需执行此代码片段 1000 次，然后观察 CPU 和内存消耗是否升高（为拒绝服务攻击创建一种简单的方法）：

var lambda = DynamicExpression
  .ParseLambda<Order, bool>("Customer=string.Format(\"{0,9999999}"+
     "{0,9999999}{0,9999999}{0,9999999}{0,9999999}\",Customer)")
  .Compile();

var arg = new Order
{
  Total = 11
};
Console.WriteLine(lambda(arg));

我写了一个 博客文章 对此。

Answer 3

只是一个想法，但是您看过 ADO.NET 数据服务吗？ 这提供了一个支持 REST 的 API，与上面的 API 非常相似，内置了许多标准 LINQ 功能。

我想不出有什么有趣的动态 LINQ 漏洞利用，但如果这是我的话，我会至少白名单成员（OrderID、CustomerName等） - 但我可能会直接编写Expression逻辑; 如果您只支持直接属性，这并不是特别困难。

例如，这里是 Where （使用您的 Contains 逻辑）：

static IQueryable<T> Where<T>(this IQueryable<T> source,
    string member, string value)
{
    var param = Expression.Parameter(typeof(T), "x");
    var arg = Expression.Constant(value, typeof(string));
    var prop = Expression.PropertyOrField(param, member);
    MethodInfo method = typeof(string).GetMethod(
        "Contains", new[] { typeof(string) });
    var invoke = Expression.Call(prop, method, arg);
    var lambda = Expression.Lambda<Func<T, bool>>(invoke, param);

    return source.Where(lambda);
}

我已经介绍了 OrderBy 之前，此处。