[ 
https://issues.apache.org/jira/browse/CASSANDRA-15556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17032249#comment-17032249
 ] 

Jordan West commented on CASSANDRA-15556:
-----------------------------------------

Thanks [~dcapwell]. Patch overall looks good a couple comments:

* I think we need to update the dependency in {{lib}} as well. 
* Minor nit: any reason I'm missing why the test can't be included in the 
existing class (it already has some unit tests)?

Also re: 

bq. it has been fixed already for sstables it seems.

I agree they are separate code paths but it looks like we are still using 
{{FastDecompressor}} in 
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/io/compress/LZ4Compressor.java#L75

> When a LZ4 stream is corrupted it could cause the JVM to crash
> --------------------------------------------------------------
>
>                 Key: CASSANDRA-15556
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15556
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: Messaging/Client
>            Reporter: David Capwell
>            Assignee: David Capwell
>            Priority: Normal
>              Labels: pull-request-available
>             Fix For: 4.0-alpha
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> This is a spin off of CASSANDRA-15313 and  CASSANDRA-15299
> This was found when lz4 sees compressed data (not all, but happens) which is 
> corrupted; in some cases the JVM crashes producing the following
> {code}
> #
> # A fatal error has been detected by the Java Runtime Environment:
> #
> #  SIGBUS (0xa) at pc=0x0000000110d46ad1, pid=86555, tid=0x0000000000001103
> #
> # JRE version: OpenJDK Runtime Environment (8.0_222-b10) (build 1.8.0_222-b10)
> # Java VM: OpenJDK 64-Bit Server VM (25.222-b10 mixed mode bsd-amd64 
> compressed oops)
> # Problematic frame:
> # C  [liblz4-java2766366422904460658.dylib+0x3ad1]  LZ4_decompress_fast+0xf1
> #
> # Failed to write core dump. Core dumps have been disabled. To enable core 
> dumping, try "ulimit -c unlimited" before starting Java again
> #
> # An error report file with more information is saved as:
> # /Users/davidcapwell/src/github/apache/cassandra/hs_err_pid86555.log
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to