Asp.net - Object reference not set to an instance of an object

Question

I am using the following to control user log in authentication. This redirects fine when the user is inactive or locked out, but when the user is deleted from the database, I get an error message...

"Object reference not set to an instance of an object" on this line...

If currentuser IsNot Nothing And currentuser.IsApproved = False Or currentuser.IsLockedOut = True Then

What could be causing this to happen?

Protected Sub Page_Init(sender As Object, e As System.EventArgs) Handles Me.Init

    If User.Identity.IsAuthenticated Then

        Dim currentuser As MembershipUser = Membership.GetUser()

        If currentuser IsNot Nothing And currentuser.IsApproved = False Or currentuser.IsLockedOut = True Then
            FormsAuthentication.SignOut()
            FormsAuthentication.RedirectToLoginPage()
        End If

        If currentuser IsNot Nothing Then
            Response.Redirect("~/media")
        End If

    End If

End Sub

Igor · Accepted Answer · 2013-08-11 01:15:19Z

1

Change the If condition to:

If currentuser IsNot Nothing Then
  If currentuser.IsApproved = False Or currentuser.IsLockedOut = True Then
    FormsAuthentication.SignOut()
    FormsAuthentication.RedirectToLoginPage()
  End If

  Response.Redirect("~/media")
End If

edited Aug 11, 2013 at 1:15

answered Aug 11, 2013 at 1:03

Igor

15.9k1 gold badge29 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Tim Schmelter · Accepted Answer · 2013-08-11 01:11:13Z

1

You need to parathesize the second part and use OrElse/AndAlso, otherwise currentuser IsNot Nothing will not be applied to all parts of your if

So this will fix it:

If currentuser IsNot Nothing AndAlso (Not currentuser.IsApproved OrElse currentuser.IsLockedOut) Then

Don't use Or (and AND) but OrElse (and AndAlso) which will short circuit. That means it will not evaluate the second part if the first part was already true. Or (and And) on the other side will evaluate both parts always.

answered Aug 11, 2013 at 1:11

Tim Schmelter

462k79 gold badges719 silver badges980 bronze badges

Comments

G.Y · Accepted Answer · 2013-08-11 01:22:23Z

1

Since 'And' receives precedence over 'Or' the expression will be evaluated as follows:

If
{ 
  currentuser IsNot Nothing And currentuser.IsApproved = False //condition 1
  Or 
  currentuser.IsLockedOut = True //condition 2
}
Then..

If current user is in fact nothing: condition 1- will not fail, but condition 2 will throw an exception since the code is trying to evaluate (nothing).somthing.

To fix the problem you'll have to add parentheses as shown:

If
  currentuser IsNot Nothing //condition 1
  And 
  (currentuser.IsApproved = False Or currentuser.IsLockedOut = True) //condition 2
Then..

Now, the 2nd condition will be evaluated only if the first condition is true.

answered Aug 11, 2013 at 1:22

G.Y

6,2092 gold badges39 silver badges58 bronze badges

Collectives™ on Stack Overflow

Asp.net - Object reference not set to an instance of an object

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related