You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by bu...@apache.org on 2015/06/20 17:22:49 UTC

[Bug 58062] New: Close IO Streams in finally statement

https://bz.apache.org/bugzilla/show_bug.cgi?id=58062

            Bug ID: 58062
           Summary: Close IO Streams in finally statement
           Product: Log4j
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: critical
          Priority: P2
         Component: Other
          Assignee: log4j-dev@logging.apache.org
          Reporter: tgttxy@163.com

The problem here is that if an exception is thrown during the read process the
method will exit without closing the stream and hence without releasing the
file system resources, it may run out of resources before it does run.I will
make a commit in github right now.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org


[Bug 58062] Close IO Streams in finally statement

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58062

Tang Xinye <tg...@163.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://github.com/apache/l
                   |                            |og4j/pull/2

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org


[Bug 58062] Close IO Streams in finally statement

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58062

Tang Xinye <tg...@163.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #1 from Tang Xinye <tg...@163.com> ---
I've made a pull request in https://github.com/apache/log4j/pull/2/files to
resolve the bug.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org


[Bug 58062] Close IO Streams in finally statement

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58062

--- Comment #2 from Gary Gregory <ga...@gmail.com> ---
Note that v1 is not actively maintained. I strongly encourage you to migrate to
v2, which includes a v1.2 compatibility layer.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org


[Bug 58062] Close IO Streams in finally statement

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=58062

Tang Xinye <tg...@163.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |---
             Status|RESOLVED                    |REOPENED

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org