Repository: incubator-ranger
Updated Branches:
  refs/heads/master ab44b6508 -> 641be5c0a


RANGER-257: fixed a code scan error from Sample Provider


Project: http://git-wip-us.apache.org/repos/asf/incubator-ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-ranger/commit/641be5c0
Tree: http://git-wip-us.apache.org/repos/asf/incubator-ranger/tree/641be5c0
Diff: http://git-wip-us.apache.org/repos/asf/incubator-ranger/diff/641be5c0

Branch: refs/heads/master
Commit: 641be5c0a4577c4f6b0af9dae434d93a7ec2f01f
Parents: ab44b65
Author: sneethiraj <[email protected]>
Authored: Fri Feb 27 08:53:44 2015 -0800
Committer: sneethiraj <[email protected]>
Committed: Fri Feb 27 08:53:44 2015 -0800

----------------------------------------------------------------------
 .../hadoop/crypto/key/SampleKeyProvider.java    | 23 ++++++++++----------
 src/main/assembly/kms.xml                       | 12 +++++-----
 2 files changed, 18 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/641be5c0/kms/src/main/java/org/apache/hadoop/crypto/key/SampleKeyProvider.java
----------------------------------------------------------------------
diff --git 
a/kms/src/main/java/org/apache/hadoop/crypto/key/SampleKeyProvider.java 
b/kms/src/main/java/org/apache/hadoop/crypto/key/SampleKeyProvider.java
index b579ebb..260b186 100644
--- a/kms/src/main/java/org/apache/hadoop/crypto/key/SampleKeyProvider.java
+++ b/kms/src/main/java/org/apache/hadoop/crypto/key/SampleKeyProvider.java
@@ -34,49 +34,51 @@ public class SampleKeyProvider extends KeyProvider {
 
        @Override
        public KeyVersion getKeyVersion(String versionName) throws IOException {
+               KeyVersion version = null ;
                // TODO Auto-generated method stub
-               return null;
+               return version ;
        }
 
        @Override
        public List<String> getKeys() throws IOException {
+               List<String> keys = null ;
                // TODO Auto-generated method stub
-               return null;
+               return keys;
        }
 
        @Override
        public List<KeyVersion> getKeyVersions(String name) throws IOException {
+               List<KeyVersion> keys = null ;
                // TODO Auto-generated method stub
-               return null;
+               return keys;
        }
 
        @Override
        public Metadata getMetadata(String name) throws IOException {
+               Metadata metadata = null ;
                // TODO Auto-generated method stub
-               
-               new KeyProvider.Metadata(null) ;
-               
-               return null;
+               return metadata ;
        }
 
        @Override
        public KeyVersion createKey(String name, byte[] material, Options 
options)
                        throws IOException {
+               KeyVersion version = null ;
                // TODO Auto-generated method stub
-               return null;
+               return version;
        }
 
        @Override
        public void deleteKey(String name) throws IOException {
                // TODO Auto-generated method stub
-               
        }
 
        @Override
        public KeyVersion rollNewVersion(String name, byte[] material)
                        throws IOException {
+               KeyVersion version = null ;
                // TODO Auto-generated method stub
-               return null;
+               return version;
        }
 
        @Override
@@ -84,7 +86,6 @@ public class SampleKeyProvider extends KeyProvider {
                // TODO Auto-generated method stub
                
        }
-
        
 
 }

http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/641be5c0/src/main/assembly/kms.xml
----------------------------------------------------------------------
diff --git a/src/main/assembly/kms.xml b/src/main/assembly/kms.xml
index 6796464..80f907c 100644
--- a/src/main/assembly/kms.xml
+++ b/src/main/assembly/kms.xml
@@ -81,26 +81,26 @@
 
        <fileSet>
                <outputDirectory>/ews/webapp/config</outputDirectory>
-               <directoryMode>0500</directoryMode>
+               <directoryMode>755</directoryMode>
                <directory>kms/config/kms-webapp</directory>
-               <fileMode>0400</fileMode>
+               <fileMode>400</fileMode>
        </fileSet>
        
        <fileSet>
                <outputDirectory>/ews/conf</outputDirectory>
-               <directoryMode>0500</directoryMode>
+               <directoryMode>755</directoryMode>
                <directory>kms/config/webserver</directory>
-               <fileMode>0400</fileMode>
+               <fileMode>400</fileMode>
        </fileSet>
 
        <fileSet>
                <outputDirectory>/scripts</outputDirectory>
-               <directoryMode>0500</directoryMode>
+               <directoryMode>755</directoryMode>
                <directory>kms/scripts</directory>
                <includes>
                        <include>ranger-*</include>
                </includes>
-               <fileMode>0544</fileMode>
+               <fileMode>544</fileMode>
        </fileSet>
 
 

Reply via email to