Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0006584
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Othersmajorhave not tried2008-12-16 15:242009-04-21 11:07
ReportershuehnerView Statuspublic 
Assigned ToHennadzi 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchpiFixed in SCM revision11385
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0006584: Potential NullpointerException

DescriptionThe File:
src/org/openbravo/authentication/basic/DefaultAuthenticationManager.java


contains the following line in the logout method:

if (request.getSession().getAttribute("#Authenticated_user") != null || !request.getSession().getAttribute("#Authenticated_user").equals("")) {

The OR condition does not make any sense here and leads to a NPE for the equals("") call, if getAttribute() really returns null. The condition should very likely be AND here.

Tagscomsup_sprint3
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0011435)
shuehner (administrator)
2008-12-16 17:14

Updated priority as this bug leads to a NPE on every logout.
(0011537)
svnbot (reporter)
2008-12-18 17:41

Repository: openbravo
Revision: 11385
Author: hennadzi
Date: 2008-12-18 17:41:14 +0100 (Thu, 18 Dec 2008)

Bug fixed 0006584: Potential NullpointerException

---
U trunk/src/org/openbravo/authentication/basic/DefaultAuthenticationManager.java
---

https://dev.openbravo.com/websvn/openbravo/?rev=11385&sc=1 [^]

- Issue History
Date Modified Username Field Change
2008-12-16 15:24 shuehner New Issue
2008-12-16 15:24 shuehner Assigned To => rafaroda
2008-12-16 15:24 shuehner sf_bug_id 0 => 2433560
2008-12-16 15:24 shuehner Regression testing => No
2008-12-16 17:14 shuehner Note Added: 0011435
2008-12-16 17:14 shuehner Priority normal => high
2008-12-16 17:14 shuehner Severity minor => major
2008-12-18 16:08 rafaroda Status new => scheduled
2008-12-18 16:08 rafaroda Assigned To rafaroda => Hennadzi
2008-12-18 16:08 rafaroda fix_in_branch => trunk
2008-12-18 17:41 svnbot Checkin
2008-12-18 17:41 svnbot Note Added: 0011537
2008-12-18 17:41 svnbot Status scheduled => resolved
2008-12-18 17:41 svnbot Resolution open => fixed
2008-12-18 17:41 svnbot svn_revision => 11385
2008-12-18 17:45 Hennadzi Tag Attached: comsup_sprint3
2009-04-21 11:07 psarobe Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker