Home/ Questions/Q 740469
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-14T08:31:10+00:00

I am a c# dev working on some code for a website in vb.net.

  • 0

I am a c# dev working on some code for a website in vb.net. We use a lot of caching on a 32bit iss 6 win 2003 box and in some cases run into OutOfMemoryException exceptions. This is the code I trace it back to and would like to know if anyone else has has this…

Public Sub CreateQueryStringNodes()
    'Check for nonstandard characters'
    Dim key As String
    Dim keyReplaceSpaces As String
    Dim r As New Regex("^[-a-zA-Z0-9_]+$", RegexOptions.Compiled)

    For Each key In HttpContext.Current.Request.Form
        If Not IsNothing(key) Then
            keyReplaceSpaces = key.Replace(" ", "_")
            If r.IsMatch(keyReplaceSpaces) Then
                CreateNode(keyReplaceSpaces, HttpContext.Current.Request(key))
            End If
        End If
    Next

    For Each key In HttpContext.Current.Request.QueryString
        If Not IsNothing(key) Then
            keyReplaceSpaces = key.Replace(" ", "_")

            If r.IsMatch(keyReplaceSpaces) Then
                CreateNode(keyReplaceSpaces, HttpContext.Current.Request(key).Replace("--", "-"))
            End If
        End If
    Next
End Sub

.NET Framework Version:2.0.50727.3053; ASP.NET Version:2.0.50727.3053

error:

Exception of type ‘System.OutOfMemoryException’ was thrown. at
Go60505(RegexRunner ) at
System.Text.RegularExpressions.CompiledRegexRunner.Go() at
System.Text.RegularExpressions.RegexRunner.Scan(Regex regex, String
text, Int32 textbeg, Int32 textend, Int32 textstart, Int32 prevlen,
Boolean quick) at System.Text.RegularExpressions.Regex.Run(Boolean
quick, Int32 prevlen, String input, Int32 beginning, Int32 length,
Int32 startat) at System.Text.RegularExpressions.Regex.IsMatch(String
input) at Xcite.Core.XML.Write.CreateQueryStringNodes() at
Xcite.Core.XML.Write..ctor(String IncludeSessionAndPostedData) at
mysite._Default.Page_Load(Object sender, EventArgs e) at
System.Web.UI.Control.OnLoad(EventArgs e) at
System.Web.UI.Control.LoadRecursive() at

thanks

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    There’s a couple of articles out there that pretty much say avoid using Compiled, it doesn’t really mean what people think it means sometimes. If I understand correctly, using Compiled actually permanently eats up memory for the duration of the application. Since you’re on the web, the application’s lifetime might be fairly long. This was supposed to be fixed/addressed in the 2.0 Framework but it looks like it hasn’t.

    • 0