Project

General

Profile

Bug #7313

cn.synchronize not closing the inputstream.

Added by Rob Nahf over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
d1_libclient_java
Target version:
Start date:
2015-08-24
Due date:
% Done:

100%

Story Points:
Sprint:

Description

testing metacat updateSystemMetadata uncovered the error in the client implementation code - the inputstream is ignored, so need to make sure it is closed in a finally block. Review other boolean-returning methods to make sure those are implemented properly, too.

Associated revisions

Revision 16144
Added by Rob Nahf over 8 years ago

refs: #7313: Fixed left-open input stream in cn.synchronize. Standardized inputSteam handling for the other method implementations (v1,v2) that return booleans, all done in finally blocks now, all now use new static method MultipartD1Node.closeLoudly.

Revision 16144
Added by Rob Nahf over 8 years ago

refs: #7313: Fixed left-open input stream in cn.synchronize. Standardized inputSteam handling for the other method implementations (v1,v2) that return booleans, all done in finally blocks now, all now use new static method MultipartD1Node.closeLoudly.

History

#1 Updated by Rob Nahf over 8 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

fixed the left-open input stream. Also refactored and standardized other boolean returning methods, to ease future maintenance.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)