You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Eduardo Aguinaga (JIRA)" <ji...@apache.org> on 2016/08/25 17:11:20 UTC
[jira] [Updated] (CASSANDRA-12538) Null Dereference
[ https://issues.apache.org/jira/browse/CASSANDRA-12538?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Eduardo Aguinaga updated CASSANDRA-12538:
-----------------------------------------
Not an issue...removed by issuer
> Null Dereference
> ----------------
>
> Key: CASSANDRA-12538
> URL: https://issues.apache.org/jira/browse/CASSANDRA-12538
> Project: Cassandra
> Issue Type: Sub-task
> Reporter: Eduardo Aguinaga
>
> Overview:
> In May through June of 2016 a static analysis was performed on version 3.0.5 of the Cassandra source code. The analysis included an automated analysis using HP Fortify v4.21 SCA and a manual analysis utilizing SciTools Understand v4. The results of that analysis includes the issue below.
> Issue:
> In the file BatchStatement.java on line 449 the object casRequest may be null and a null dereference exception could be generated. casRequest is initialized to null on linw 412 and if the for loop beginning on line 415 is skipped the null dereference will occur.
> {code:java}
> BatchStatement.java, lines 412-450:
> 412 CQL3CasRequest casRequest = null;
> 413 Set<ColumnDefinition> columnsWithConditions = new LinkedHashSet<>();
> 414
> 415 for (int i = 0; i < statements.size(); i++)
> 416 {
> 417 ModificationStatement statement = statements.get(i);
> 418 QueryOptions statementOptions = options.forStatement(i);
> 419 long timestamp = attrs.getTimestamp(now, statementOptions);
> 420 List<ByteBuffer> pks = statement.buildPartitionKeyNames(statementOptions);
> 421 if (pks.size() > 1)
> 422 throw new IllegalArgumentException("Batch with conditions cannot span multiple partitions (you cannot use IN on the partition key)");
> 423 if (key == null)
> 424 {
> 425 key = statement.cfm.decorateKey(pks.get(0));
> 426 casRequest = new CQL3CasRequest(statement.cfm, key, true, conditionColumns, updatesRegularRows, updatesStaticRow);
> 427 }
> 428 else if (!key.getKey().equals(pks.get(0)))
> 429 {
> 430 throw new InvalidRequestException("Batch with conditions cannot span multiple partitions");
> 431 }
> 432
> ...
> 448 }
> 449 casRequest.addRowUpdate(clustering, statement, statementOptions, timestamp);
> 450 }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)