Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-4293

Mutable public static variables

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Security

    Description

      There are quite a few instances of public static fields which appear to be intended to be constant, but which are not final.

      The code would be safer if all the constant fields were made final.

      Patch to follow for some of the problem fields.

      The following also ought to be fixed:

      org.apache.derby.iapi.services.property.PropertyUtil.servicePropertyList (e.g. use an accessor to lookup the values)
      org.apache.derby.iapi.types.JSQLType.primitiveNames (ditto)

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            knutanders Knut Anders Hatlen
            sebb Sebb
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment