From 4c451f44ed9c19373a2657b9f87900880dfb8e51 Mon Sep 17 00:00:00 2001
From: Dominik Stadler <centic@apache.org>
Date: Wed, 26 Nov 2014 18:39:44 +0000
Subject: [PATCH] Add more output in case of unknown cipher-ids to aid in
 debugging bugs like 57195

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1641883 13f79535-47bb-0310-9956-ffa450edef68
---
 .../poi/poifs/crypt/CipherAlgorithm.java      |  6 +--
 .../poi/poifs/crypt/TestCipherAlgorithm.java  | 42 +++++++++++++++++++
 2 files changed, 44 insertions(+), 4 deletions(-)
 create mode 100644 src/testcases/org/apache/poi/poifs/crypt/TestCipherAlgorithm.java

diff --git a/src/java/org/apache/poi/poifs/crypt/CipherAlgorithm.java b/src/java/org/apache/poi/poifs/crypt/CipherAlgorithm.java
index 68682f496..cee49d2be 100644
--- a/src/java/org/apache/poi/poifs/crypt/CipherAlgorithm.java
+++ b/src/java/org/apache/poi/poifs/crypt/CipherAlgorithm.java
@@ -64,7 +64,7 @@ public enum CipherAlgorithm {
         for (CipherAlgorithm ca : CipherAlgorithm.values()) {
             if (ca.ecmaId == ecmaId) return ca;
         }
-        throw new EncryptedDocumentException("cipher algorithm not found");
+        throw new EncryptedDocumentException("cipher algorithm " + ecmaId + " not found");
     }
     
     public static CipherAlgorithm fromXmlId(String xmlId, int keySize) {
@@ -74,8 +74,6 @@ public enum CipherAlgorithm {
                 if (ks == keySize) return ca;
             }
         }
-        throw new EncryptedDocumentException("cipher algorithm not found");
+        throw new EncryptedDocumentException("cipher algorithm " + xmlId + "/" + keySize + " not found");
     }
-    
-
 }
\ No newline at end of file
diff --git a/src/testcases/org/apache/poi/poifs/crypt/TestCipherAlgorithm.java b/src/testcases/org/apache/poi/poifs/crypt/TestCipherAlgorithm.java
new file mode 100644
index 000000000..814620b82
--- /dev/null
+++ b/src/testcases/org/apache/poi/poifs/crypt/TestCipherAlgorithm.java
@@ -0,0 +1,42 @@
+package org.apache.poi.poifs.crypt;
+
+import static org.junit.Assert.*;
+
+import org.apache.poi.EncryptedDocumentException;
+import org.junit.Test;
+
+public class TestCipherAlgorithm {
+    @Test
+    public void test() {
+        assertEquals(128, CipherAlgorithm.aes128.defaultKeySize);
+        
+        for(CipherAlgorithm alg : CipherAlgorithm.values()) {
+            assertEquals(alg, CipherAlgorithm.valueOf(alg.toString()));
+        }
+
+        assertEquals(CipherAlgorithm.aes128, CipherAlgorithm.fromEcmaId(0x660E));
+        assertEquals(CipherAlgorithm.aes192, CipherAlgorithm.fromXmlId("AES", 192));
+        
+        try {
+            CipherAlgorithm.fromEcmaId(0);
+            fail("Should throw exception");
+        } catch (EncryptedDocumentException e) {
+            // expected
+        }
+
+        try {
+            CipherAlgorithm.fromXmlId("AES", 1);
+            fail("Should throw exception");
+        } catch (EncryptedDocumentException e) {
+            // expected
+        }
+
+        try {
+            CipherAlgorithm.fromXmlId("RC1", 0x40);
+            fail("Should throw exception");
+        } catch (EncryptedDocumentException e) {
+            // expected
+        }
+    }
+
+}