ditaoctavia / pwm

Automatically exported from code.google.com/p/pwm
0 stars 0 forks source link

NewUserServlet null pointer exception #640

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Setup the new user registration with a mail field that is not required
2. Add a Mobile Phone number field that is required
3. Disable the e-mail verification of the new user registration module
4. Enable the sms verification of the new user registration module

What is the expected output? What do you see instead?
I would expect that PWM skips the new user email, because there is no email 
address. Instead I get a null pointer exception. PWM does not check correctly 
wether the email address is there.

Attached a patch for version 1.7.1. This issue is relevant for trunk as well, 
though that patch should be a bit different (check is done on 
configuredEmailSetting, but also misses the check on no email address which can 
lead to a null pointer exception)

Original issue reported on code.google.com by sebastia...@gmail.com on 27 Nov 2014 at 11:25

Attachments: