Close the digest stream when shaDigestForFile can't read a file

[pjfanning]

BaseSchemaResourceManager.shaDigestForFile closes the stream only after the read loop completes:

DigestInputStream str = digestInputStream(inputStreamForFile(filename));

byte[] dummy = new byte[4096];
int i = 1;
while (i > 0) {
    i = str.read(dummy);
}

str.close();

An IOException part-way through skips the close. The caller, syncCacheWithLocalXsdFiles, swallows it —

} catch (IOException e) {
    // unable to read digest... no problem, ignore then
}

— and carries on to the next file, so sdownload -sync over a cache containing unreadable .xsd files leaks one handle per file instead of failing.

Wrapped in a try-with-resources. getMessageDigest().digest() moves inside the block; it reads the digest state and does not require the stream to be closed first.

Added SchemaResourceManagerStreamTest, which serves a stream that opens and then fails every read. It fails on trunk and passes with this change. Full suite: 3171 tests pass (170 skipped).

🤖 Generated with Claude Code