Corrigir o escape de identificadores delimitados no SQL Server sem usar QUOTENAME

Seu QuoteName A função precisa verificar o comprimento, porque a função T-SQL QUOTENAME especifica o comprimento máximo que ela retorna. Usando seu exemplo:

String.Format(@"declare @delimitedIdentifier nvarchar(258);
set @delimitedIdentifier = {0};", QuoteName(identifier));

Se QuoteName(identifier) tiver mais de 258 caracteres, será truncado silenciosamente quando atribuído a @delimitedIdentifier . Quando isso acontece, você abre a possibilidade de @delimitedIdentifier escapar indevidamente.

Existe um Artigo do MSDN por Bala Neerumalla, um "desenvolvedor de software de segurança da Microsoft", que explica o assunto com mais profundidade. O artigo também contém a coisa mais próxima que encontrei da "documentação definitiva sobre como escapar de identificadores citados no SQL Server":

Este é o código C# que estou usando atualmente:

/// <summary>
/// Returns a string with the delimiters added to make the input string
/// a valid SQL Server delimited identifier. Brackets are used as the
/// delimiter. Unlike the T-SQL version, an ArgumentException is thrown
/// instead of returning a null for invalid arguments.
/// </summary>
/// <param name="name">sysname, limited to 128 characters.</param>
/// <returns>An escaped identifier, no longer than 258 characters.</returns>
public static string QuoteName(string name) { return QuoteName(name, '['); }

/// <summary>
/// Returns a string with the delimiters added to make the input string
/// a valid SQL Server delimited identifier. Unlike the T-SQL version,
/// an ArgumentException is thrown instead of returning a null for
/// invalid arguments.
/// </summary>
/// <param name="name">sysname, limited to 128 characters.</param>
/// <param name="quoteCharacter">Can be a single quotation mark ( ' ), a
/// left or right bracket ( [] ), or a double quotation mark ( " ).</param>
/// <returns>An escaped identifier, no longer than 258 characters.</returns>
public static string QuoteName(string name, char quoteCharacter) {
    name = name ?? String.Empty;
    const int sysnameLength = 128;
    if (name.Length > sysnameLength) {
        throw new ArgumentException(String.Format(
            "name is longer than {0} characters", sysnameLength));
    }
    switch (quoteCharacter) {
        case '\'':
            return String.Format("'{0}'", name.Replace("'", "''"));
        case '"':
            return String.Format("\"{0}\"", name.Replace("\"", "\"\""));
        case '[':
        case ']':
            return String.Format("[{0}]", name.Replace("]", "]]"));
        default:
            throw new ArgumentException(
                "quoteCharacter must be one of: ', \", [, or ]");
    }
}