2

我有使用指针对数据块进行异或的代码,这很快,但我想摆脱对程序集的“不安全”要求。如果我将其更改为使用 LayoutKind.Explicit 并在“byte []”之上覆盖“ulong []”,我基本上会做与指针相同的事情,但它似乎同样危险。这两者的主要区别在于“安全”版本的速度大约是“不安全”版本的 1/2。

这是解决“不安全”程序集的合法方式,还是一次访问 byte[] 1 个字节是以安全方式执行此操作的唯一合法方式?

private unsafe static void UnsafeEncode(
    byte[] buffer, int bufPos, ulong[] vector, int SectionLength, byte vectorIndex)
{
    fixed (byte* p = &buffer[bufPos])
    {
        ulong* pCur = (ulong*)p;
        ulong* pEnd = pCur + SectionLength;
        while (pCur < pEnd)
        {
            *pCur ^= vector[vectorIndex++];
            pCur++;
        }
    }
}

[StructLayout(LayoutKind.Explicit)]
private struct ArrayOverlay
{
    [FieldOffset(0)]
    public byte[] Bytes;
    [FieldOffset(0)]
    public ulong[] Longs;
}

private static void SafeEncode(
    byte[] buffer, int bufPos, ulong[] vector, int SectionLength, byte vectorIndex)
{
    var overlay = new ArrayOverlay { Bytes = buffer };
    int shiftleft = (bufPos & 7) << 3;
    int pos = bufPos >> 3;
    if (shiftleft == 0)
    {
        for (int i = 0; i < SectionLength; i++)
            overlay.Longs[i + pos] ^= vector[vectorIndex++];
    }
    else
    {
        int shiftright = (64 - shiftleft) & 63;
        ulong oldVec = 0;
        for (int i = 0; i < SectionLength; i++)
        {
            var vec = vector[vectorIndex++];
            overlay.Longs[i + pos] ^= (vec << shiftleft) | (oldVec >> shiftright);
            oldVec = vec;
        }
        overlay.Longs[SectionLength + pos] ^= (oldVec >> shiftright);
    }
}
4

1 回答 1

1

这是解决“不安全”程序集的合法方式,还是一次访问 byte[] 1 个字节是以安全方式执行此操作的唯一合法方式?

这是合法的,因为它 1) 有效,并且 2) 可以产生正确的结果。

诚然,它不一定是“安全的”,因为您可以通过这样做来做“讨厌”的事情,但它仍然会起作用。在这种情况下,您肯定违反了安全代码的精神,因为您可以做“安全”代码中不应允许的事情,例如有效地绕过数组长度检查并导致缓冲区溢出。

一次访问一个元素的数组更符合“安全”代码的精神,但有时使用这样的技巧很有价值,因为它在需要时提供了一种性能机制。如果您的特定场景需要性能,并且正常机制无法充分执行,这可能是一种使代码“工作”的方法,因为它符合您的规范(性能要求),而不会违反标记的安全约束组装为不安全。

于 2013-06-06T16:31:44.897 回答