You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@camel.apache.org by BruceKuiLiu <gi...@git.apache.org> on 2017/10/12 09:36:14 UTC

[GitHub] camel pull request #2031: Add the "final" modifier to a public static field.

GitHub user BruceKuiLiu opened a pull request:

    https://github.com/apache/camel/pull/2031

    Add the "final" modifier to a public static field.

    This static field public but not final, and could be changed by malicious code or by accident from another package. The field could be made final to avoid this vulnerability.
    http://findbugs.sourceforge.net/bugDescriptions.html#MS_SHOULD_BE_FINAL

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/BruceKuiLiu/camel ShouldBeFinal

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/2031.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2031
    
----
commit efc274bf140882ebde2055211651e916d9f8aee8
Author: Kui LIU <br...@gmail.com>
Date:   2017-10-12T09:34:21Z

    Add the "final" modifier to a public static field.
    
    This static field public but not final, and could be changed by malicious code or by accident from another package. The field could be made final to avoid this vulnerability.
    http://findbugs.sourceforge.net/bugDescriptions.html#MS_SHOULD_BE_FINAL

----


---

[GitHub] camel pull request #2031: Add the "final" modifier to a public static field.

Posted by BruceKuiLiu <gi...@git.apache.org>.

Github user BruceKuiLiu closed the pull request at:

    https://github.com/apache/camel/pull/2031


---