NullReferenceException, för vissa

Börja med att ta reda på vad som är null.

emission skrev:

Börja med att ta reda på vad som är null.

Precis vad jag hade gjort om jag kunde återskapa problemet. Det som förvirrar mig är att det skiljer sig beroende på vilken dator hon använder, och hur det kan skapa ett applikationsfel i websajten.

Felet kommer vid OnClick på submitknappen, men jag kan inte se att någonting i den skulle kunna bli null... och framförallt att det skulle skilja sig mellan vilka system man använder.

Vide skrev:

Hon kör IE7, cookies och javascript är aktiverat och krävs. Längst ned i hennes IE7 så finns en ikon (ett vitt fönster med ett utropstecken), denna vet jag inte vad den gör.

Skjuter lite i mörkret här...
Brukar inte dom ikonerna symbolisera Javascript-errors? Går det att dubbelklicka på ikonen? Bör inte vara det, men jag har själv varit med om javascript-errors på olika datorer som dock har samma konfiguration.

Vide skrev:

Hon kör IE7, cookies och javascript är aktiverat och krävs. Längst ned i hennes IE7 så finns en ikon (ett vitt fönster med ett utropstecken), denna vet jag inte vad den gör.

Det är phishing-skyddet i IE7 som ser ut så... Men jag tror inte denna har med saken att göra.

Vide skrev:

Precis vad jag hade gjort om jag kunde återskapa problemet. Det som förvirrar mig är att det skiljer sig beroende på vilken dator hon använder, och hur det kan skapa ett applikationsfel i websajten.

Felet kommer vid OnClick på submitknappen, men jag kan inte se att någonting i den skulle kunna bli null... och framförallt att det skulle skilja sig mellan vilka system man använder.

Det är väl "bara" att lägga till lite kod för att logga felet?

Kan du dela med dig av OnClick-handlern?

emission skrev:

Kan du dela med dig av OnClick-handlern?

Här är den:

  public void loginUser( object sender, EventArgs e )
  {
    string _password = passwordTextBox.Text;

    if ( terms.Checked )
    {
      if ( true == aglobals.User.loadUser( false, "", true, _password ) )
      {
        string _strQuery;
        
        _strQuery = 
          "insert into xyz" +
          " (login_id,ipaddress,referer)" +
          " values(" +
          "'" + aglobals.User.getValueAsInt32( "login_id" ) + "'," +
          "'" + Request.ServerVariables[ "REMOTE_HOST" ].ToString() + "'," +
          "'" + Request.ServerVariables[ "HTTP_REFERER" ].ToString() + "')";
          
        aglobals.queryDB( _strQuery );
          
          FormsAuthentication.SetAuthCookie( "user", false );
          Response.Redirect( "main.aspx" );
      }
      else
      {
        if( aglobals.User.getErrCode() == 1 )
        {
          errormsgLiteral.Text = "<br />" + aglobals.User.getErrMsg();
        }
        else
        {
          errormsgLiteral.Text = "<br />The password you entered was wrong. Please try again.";
        }
      }
    }
    else
      errormsgLiteral.Text = "<br />You must accept the terms and conditions.";
  }

aglobals är ett globalt objekt som alltid initialiseras.

Saker som skulle kunna vara null:

passwordTextBox
terms
aglobals
aglobals.User
errormsgLiteral
Request.ServerVariables[ "REMOTE_HOST" ]
Request.ServerVariables[ "HTTP_REFERER" ]

Av dessa är det HTTP_REFERER som är den mest sannolika boven. Skippa ToString(). Den är nämligen redan en sträng, vilket även gäller REMOTE_HOST.

I övrigt bör du använda parametriserade frågor. Rent teoretiskt är sidan över för HTTP header SQL Injection.

passwordTextBox, terms och errormsgLiteral är serverside-objekt i aspsidan, kan dom rent teoretiskt vara null då?

aglobals och aglobals.User initieras alltid i page_load.

Request.ServerVariables[ "REMOTE_HOST" ] och Request.ServerVariables[ "HTTP_REFERER" ] kan visserligen vara null, skall testa att kontrollera dessa...

Av dessa är det HTTP_REFERER som är den mest sannolika boven. Skippa ToString(). Den är nämligen redan en sträng, vilket även gäller REMOTE_HOST.

Jag testade att skriva in adressen rakt av, men den blev inte null. Finns det något annat sätt som den blir null av?

Men jag håller med, det borde rimligtvis bara kunna vara servervariablerna som skulle kunna bli null, annars problemet uppstått hos flera användare.

I övrigt bör du använda parametriserade frågor. Rent teoretiskt är sidan över för HTTP header SQL Injection.

Tack, bra tips!

Vide skrev:

Jag testade att skriva in adressen rakt av, men den blev inte null. Finns det något annat sätt som den blir null av?

Det är inte alls orimligt att webbläsaren inte skickar med HTTP_REFERER.

emission skrev:

Det är inte alls orimligt att webbläsaren inte skickar med HTTP_REFERER.

Det stämmer, för om du öppnar din webläsare och skriver in en url direkt på sidan, så har man ingen HTTP_REFERER eftersom man inte kommer från någon sida. Men läsare kanske är så smart att den själv lägger till den information eftersom den vet vilken sida du är på. Det är dock inget som jag har hört något om... och stöds garanterat inte av några standards.

- M

Gängse standard i webbläsare är att HTTP_REFERER inte skickas med, annat än när man klickar på en länk eller när t.ex. en bild hämtas på en sida. Webbläsare/proxyservrar etc. kan välja att skicka med dem ännu mer sällan, men att skicka med den mer ofta vore ett brott mot HTTP-standarden.