Bug 926050

Summary: Constant fields that are not final
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Brian Stansberry <brian.stansberry>
Component: ServerAssignee: Brian Stansberry <brian.stansberry>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: jason.greene, lthon, myarboro, rsvoboda
Target Milestone: ER7   
Target Release: EAP 6.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-15 16:17:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
list of non-final constants none

Description Brian Stansberry 2013-03-23 13:05:46 UTC
Created attachment 715073 [details]
list of non-final constants

Analysis by Stephan Mueller shows that the AS codebase includes a significant number of public constant fields that are missing the "final" keyword, which could allow incorrect or nefarious code to change them. This task is to fix these.
As part of the work for a few cases I'll reduce the field visibility.

Comment 3 Rostislav Svoboda 2013-04-25 14:10:45 UTC
PR is ready - https://github.com/jbossas/jboss-eap/pull/73

setting blocker flag to get this change to EAP 6.1.0

Comment 6 Rostislav Svoboda 2013-04-29 07:02:53 UTC
Removing blocker flag based on Brian's comment.

Comment 13 Ladislav Thon 2013-11-14 14:30:52 UTC
Verified with EAP 6.2.0.CR1. Only checked that the commit is present in the CR1 tag and that some of the changes from the commit are actually in the CR1-src.zip.