User shouldn't really be a structure. With a size that big, you would be stuffing a whole lot of bytes onto the stack if you ever passed a User as an argument to a method.

However, that's not the issue here. The code you showed doesn't seem to cover the question. You create an image in UserImage, but the only time you assign to an Image property you are assigning AdminImage. Where did that come from?

You probably don't need to go through the step of creating the image. TG mentioned that the PB has a method to load from a path. It's this one:

https://msdn.microsoft.com/en-us/lib...v=vs.110).aspx

So, you can probably just use that and bypass the need to create an image. Haven't tried it, though, cause I didn't know it existed.