PR #509: Make PutNextEntry throw if the entry is AES and no password …

…has been set

refs #507

src/ICSharpCode.SharpZipLib/Zip/ZipOutputStream.cs

@@ -262,6 +262,12 @@ public void PutNextEntry(ZipEntry entry)
 				throw new NotImplementedException("Compression method not supported");
 			}
 
+			// A password must have been set in order to add AES encrypted entries
+			if (entry.AESKeySize > 0 && string.IsNullOrEmpty(this.Password))
+			{
+				throw new InvalidOperationException("The Password property must be set before AES encrypted entries can be added");
+			}
+
 			int compressionLevel = defaultCompressionLevel;
 
 			// Clear flags that the library manages internally

test/ICSharpCode.SharpZipLib.Tests/Zip/StreamHandling.cs

@@ -502,5 +502,23 @@ public void ShouldBeAbleToReadEntriesWithInvalidFileNames()
 				}
 			}
 		}
+
+		/// <summary>
+		/// Test for https://github.com/icsharpcode/SharpZipLib/issues/507
+		/// </summary>
+		[Test]
+		[Category("Zip")]
+		public void AddingAnAESEntryWithNoPasswordShouldThrow()
+		{
+			using (var memoryStream = new MemoryStream())
+			{
+				using (var outStream = new ZipOutputStream(memoryStream))
+				{
+					var newEntry = new ZipEntry("test") { AESKeySize = 256 };
+
+					Assert.Throws<InvalidOperationException>(() => outStream.PutNextEntry(newEntry));
+				}
+			}
+		}
 	}
 }