Glavo edited a comment on pull request #248:
URL: https://github.com/apache/commons-compress/pull/248#issuecomment-1038543634


   > Said the above, looks like the Compress class itself is marked as not 
thread-safe as well @Glavo ?
   > 
   > See the `@NotThreadSafe` annotation. So we could have other issues besides 
the `CharsetEncoder` that would have to be fixed.
   
   I checked the 7z source code and didn't see a similar problem. 
   
   My goal is not to make a instance of `SevenZFile` thread safe. I have reason 
to believe that `@NotThreadSafe` indicates that a single instance cannot 
operate in parallel, and the problem I'm dealing with now is security in the 
global scope. 
   
    `utf16Decode` is used to initialize the instance of `SevenZFile`, this is 
reflected in the fact that errors may occur if multiple `SevenZFile` instances 
with Unicode extended flat characters in passwords are created in parallel. It 
doesn't appear on one instance, so I want to solve it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to