cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Eduardo Aguinaga (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CASSANDRA-12308) Use of Dynamic Class Loading, Use of Externally-Controlled Input to Select Classes or Code
Date Tue, 26 Jul 2016 18:22:20 GMT

     [ https://issues.apache.org/jira/browse/CASSANDRA-12308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Eduardo Aguinaga updated CASSANDRA-12308:
-----------------------------------------
    Description: 
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:
Dynamically loaded code has the potential to be malicious. The application uses external input
to select which classes or code to use, but it does not sufficiently prevent the input from
selecting improper classes or code.

The snippet below shows the issue which ends on line 585 by instantiating a class by name.
ConfigHelper.java, lines 558-591:
{code:java}
558 @SuppressWarnings("resource")
559 public static Cassandra.Client createConnection(Configuration conf, String host, Integer
port) throws IOException
560 {
561     try
562     {
563         TTransport transport = getClientTransportFactory(conf).openTransport(host, port);
564         return new Cassandra.Client(new TBinaryProtocol(transport, true, true));
565     }
566     catch (Exception e)
567     {
568         throw new IOException("Unable to connect to server " + host + ":" + port, e);
569     }
570 }
571 
572 public static ITransportFactory getClientTransportFactory(Configuration conf)
573 {
574     String factoryClassName = conf.get(ITransportFactory.PROPERTY_KEY, TFramedTransportFactory.class.getName());
575     ITransportFactory factory = getClientTransportFactory(factoryClassName);
576     Map<String, String> options = getOptions(conf, factory.supportedOptions());
577     factory.setOptions(options);
578     return factory;
579 }
580 
581 private static ITransportFactory getClientTransportFactory(String factoryClassName)
582 {
583     try
584     {
585         return (ITransportFactory) Class.forName(factoryClassName).newInstance();
586     }
587     catch (Exception e)
588     {
589         throw new RuntimeException("Failed to instantiate transport factory:" + factoryClassName,
e);
590     }
591 }
{code}

  was:
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:
Dynamically loaded code has the potential to be malicious. The application uses external input
with reflection to select which classes or code to use, but it does not sufficiently prevent
the input from selecting improper classes or code.

The snippet below shows the issue which ends on line 585 by instantiating a class by name.
ConfigHelper.java, lines 558-591:
{code:java}
558 @SuppressWarnings("resource")
559 public static Cassandra.Client createConnection(Configuration conf, String host, Integer
port) throws IOException
560 {
561     try
562     {
563         TTransport transport = getClientTransportFactory(conf).openTransport(host, port);
564         return new Cassandra.Client(new TBinaryProtocol(transport, true, true));
565     }
566     catch (Exception e)
567     {
568         throw new IOException("Unable to connect to server " + host + ":" + port, e);
569     }
570 }
571 
572 public static ITransportFactory getClientTransportFactory(Configuration conf)
573 {
574     String factoryClassName = conf.get(ITransportFactory.PROPERTY_KEY, TFramedTransportFactory.class.getName());
575     ITransportFactory factory = getClientTransportFactory(factoryClassName);
576     Map<String, String> options = getOptions(conf, factory.supportedOptions());
577     factory.setOptions(options);
578     return factory;
579 }
580 
581 private static ITransportFactory getClientTransportFactory(String factoryClassName)
582 {
583     try
584     {
585         return (ITransportFactory) Class.forName(factoryClassName).newInstance();
586     }
587     catch (Exception e)
588     {
589         throw new RuntimeException("Failed to instantiate transport factory:" + factoryClassName,
e);
590     }
591 }
{code}


> Use of Dynamic Class Loading, Use of Externally-Controlled Input to Select Classes or
Code
> ------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-12308
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12308
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Eduardo Aguinaga
>             Fix For: 3.0.5
>
>
> 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:
> Dynamically loaded code has the potential to be malicious. The application uses external
input to select which classes or code to use, but it does not sufficiently prevent the input
from selecting improper classes or code.
> The snippet below shows the issue which ends on line 585 by instantiating a class by
name.
> ConfigHelper.java, lines 558-591:
> {code:java}
> 558 @SuppressWarnings("resource")
> 559 public static Cassandra.Client createConnection(Configuration conf, String host,
Integer port) throws IOException
> 560 {
> 561     try
> 562     {
> 563         TTransport transport = getClientTransportFactory(conf).openTransport(host,
port);
> 564         return new Cassandra.Client(new TBinaryProtocol(transport, true, true));
> 565     }
> 566     catch (Exception e)
> 567     {
> 568         throw new IOException("Unable to connect to server " + host + ":" + port,
e);
> 569     }
> 570 }
> 571 
> 572 public static ITransportFactory getClientTransportFactory(Configuration conf)
> 573 {
> 574     String factoryClassName = conf.get(ITransportFactory.PROPERTY_KEY, TFramedTransportFactory.class.getName());
> 575     ITransportFactory factory = getClientTransportFactory(factoryClassName);
> 576     Map<String, String> options = getOptions(conf, factory.supportedOptions());
> 577     factory.setOptions(options);
> 578     return factory;
> 579 }
> 580 
> 581 private static ITransportFactory getClientTransportFactory(String factoryClassName)
> 582 {
> 583     try
> 584     {
> 585         return (ITransportFactory) Class.forName(factoryClassName).newInstance();
> 586     }
> 587     catch (Exception e)
> 588     {
> 589         throw new RuntimeException("Failed to instantiate transport factory:" + factoryClassName,
e);
> 590     }
> 591 }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message