Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0006584 | Openbravo ERP | Z. Others | public | 2008-12-16 15:24 | 2009-04-21 11:07 |
|
| Reporter | shuehner | |
| Assigned To | Hennadzi | |
| Priority | high | Severity | major | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | pi | |
| Target Version | | Fixed in Version | | |
| Merge Request Status | |
| Review Assigned To | |
| OBNetwork customer | |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0006584: Potential NullpointerException |
| Description | The 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.
|
| Steps To Reproduce | |
| Proposed Solution | |
| Additional Information | |
| Tags | comsup_sprint3 |
| Relationships | |
| Attached Files | |
|
| 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 |