Was bedeutet ---- im Zusammenhang mit StringBuilder.ToString()?

Question

Die Reference Source page for stringbuilder.cs hat diesen Kommentar im ToString Methode:

if (chunk.m_ChunkLength > 0) 
{ 
    // Copy these into local variables so that they 
    // are stable even in the presence of ----s (hackers might do this) 
    char[] sourceArray = chunk.m_ChunkChars; 
    int chunkOffset = chunk.m_ChunkOffset; 
    int chunkLength = chunk.m_ChunkLength; 

Was bedeutet das? Ist ----s etwas, das ein böswilliger Benutzer in eine zu formatierende Zeichenfolge einfügen könnte?

Answer 1

Im CoreCLR Repository Ihnen einen volleren Zitat haben:

kopieren diese in lokale Variablen, so dass sie

Github

Grundsätzlich stabil, auch in Gegenwart von Rennbedingungen sind: es ist eine Threading-Überlegung.

Answer 2

Denken Sie nicht, dass dies der Fall ist - der fragliche Code wird in lokale Variablen kopiert, um zu verhindern, dass schlimme Dinge passieren, wenn die String-Builder-Instanz in einem anderen Thread mutiert ist.

Ich denke, die auf ein Wort mit vier Buchstaben ...

Zusätzlich

Answer 3

von @Jeroen auf die große Antwort schwören beziehen, das mehr als nur ein Einfädeln Überlegung. Es soll verhindern, dass jemand absichtlich eine Wettlaufsituation erzeugt und auf diese Weise einen Pufferüberlauf verursacht. Später im Code wird die Länge dieser lokalen Variablen überprüft. Wenn der Code die Länge der zugänglichen Variablen überprüfen war stattdessen könnte es auf einem anderen Thread geändert hat zwischen wurde die Zeitdauer geprüft und wstrcpy hieß:

 // Check that we will not overrun our boundaries. 
     if ((uint)(chunkLength + chunkOffset) <= ret.Length && (uint)chunkLength <= (uint)sourceArray.Length) 
     { 
      /// 
      /// imagine that another thread has changed the chunk.m_ChunkChars array here! 
      /// we're now in big trouble, our attempt to prevent a buffer overflow has been thawrted! 
      /// oh wait, we're ok, because we're using a local variable that the other thread can't access anyway. 
      fixed (char* sourcePtr = sourceArray) 
       string.wstrcpy(destinationPtr + chunkOffset, sourcePtr, chunkLength); 
     } 
     else 
     { 
      throw new ArgumentOutOfRangeException("chunkLength", Environment.GetResourceString("ArgumentOutOfRange_Index")); 
     } 
    } 
    chunk = chunk.m_ChunkPrevious; 
} while (chunk != null); 

wirklich interessante Frage though.

Answer 4

Der Quellcode für die veröffentlichte Referenzquelle wird durch einen Filter geleitet, der anstößige Inhalte aus der Quelle entfernt. Verboten Worte sind eins, Microsoft-Programmierer use profanity in ihren Kommentaren. So sind die Namen der Entwickler, Microsoft will ihre Identität verbergen. Ein solches Wort oder ein solcher Name wird durch Bindestriche ersetzt.

In diesem Fall können Sie erkennen, was früher aus der CoreCLR, der Open-Source-Version von .NET Framework, da war. Es ist ein Wort verboten:

// diese in lokale Variablen kopieren, so dass sie auch in Gegenwart von Rennen Bedingungen

stabil sind, die von der ursprünglichen von Hand bearbeitet wurde, dass Sie sah Bevor sie an Github übergeben wird, will Microsoft auch nicht ihre Kunden beschuldigen, Hacker zu sein, hieß es ursprünglich races und verwandelte sich in ----s :)