CryptographicEx ception when closing XmlTextReader

[Tim Barg]

I am using a CryptoStream to encrypt and then save an xml file. However,
when reading it back in, I am getting a CryptographicEx ception. I am hitting
this because I have some large xml files ( > 10MB) and I only need a certain
node...I use an XmlTextReader to find the node I need, and then attempt to
abort reading in the rest of the xml, but when I close the XmlTextReader I
get the exception. I've included some sample code and xml that should
exhibit the behavior.

Note: If I use Skip() to get to the end of the xml, then I don't get
exceptions, but as I mentioned, the files I'm dealing with can be large and
Skip() takes quite a while to skip over nodes that I don't care about which
really hurts performance. Any ideas?

--------- SAMPLE SOURCE CODE ---------

using System;
using System.Security .Cryptography;
using System.Text;
using System.IO;
using System.Xml;

private const string MY_PASSWORD = "test"; private const string
SOURCE_XML_FILE _PATH = @"C:\Sample.xml "; private const string
ENCRYPTED_FILE_ PATH = @"C:\Sample.xml e";

static void Main(string[] args)
{
EncryptFile(SOU RCE_XML_FILE_PA TH, ENCRYPTED_FILE_ PATH, MY_PASSWORD);
ReadEncryptedXm l(ENCRYPTED_FIL E_PATH, MY_PASSWORD); }

private static void ReadEncryptedXm l(string encryptedFilePa th, string
password) {
FileStream fileStreamSrc = null;
CryptoStream cstream = null;
StreamReader streamReader = null;
XmlTextReader xmlReader = null;

try {
fileStreamSrc = new FileStream(encr yptedFilePath, FileMode.Open,
FileAccess.Read );
cstream = CreateEncrypted Stream(fileStre amSrc, password,
CryptoStreamMod e.Read);

streamReader = new StreamReader(cs tream, Encoding.UTF8);
xmlReader = new XmlTextReader(s treamReader);

xmlReader.White spaceHandling = WhitespaceHandl ing.None;
xmlReader.MoveT oContent();

bool breakOut = false;
while (xmlReader.Node Type != XmlNodeType.End Element) {
switch (xmlReader.Name ) {
case "States":
xmlReader.Read( );
break;
case "State":
xmlReader.Read( );
break;
case "Name":
xmlReader.Skip( );
break;
case "Population ":
XmlDocument retValue1 = new
XmlDocument();
retValue1.Appen dChild(
retValue1.ReadN ode( xmlReader ));
breakOut = true;
break;
default:
xmlReader.Skip( );
break;
}

if (breakOut) {
break;
}
}

xmlReader.Close ();
streamReader.Cl ose();
cstream.Close() ;
fileStreamSrc.C lose();
}
catch (CryptographicE xception ex) {
Console.WriteLi ne(ex.ToString( ));
}
catch (Exception ex) {
Console.WriteLi ne(ex.ToString( ));
}
}

private static void EncryptFile(str ing sourceXmlFilePa th, string
destEncryptedFi lePath, string password) {
FileStream fileStreamSrc = null;
FileStream fileStreamDest = null;
CryptoStream cstream = null;

try {
fileStreamSrc = new FileStream(sour ceXmlFilePath, FileMode.Open,
FileAccess.Read );
fileStreamDest = new FileStream(dest EncryptedFilePa th, FileMode.Create ,
FileAccess.Writ e);

long rdlen = 0;
long totlen = fileStreamSrc.L ength;
int len;

cstream = CreateEncrypted Stream(fileStre amDest, password,
CryptoStreamMod e.Write);

// Read from input file, encrypt and write to output file
byte[] b = new byte[1024];
while(rdlen < totlen) {
len = fileStreamSrc.R ead(b, 0, b.Length);
cstream.Write(b , 0, len);
rdlen = rdlen + len;
}

cstream.Close() ;
fileStreamDest. Close();
fileStreamSrc.C lose();
}
catch (Exception ex) {
Console.WriteLi ne(ex.ToString( ));
}
}

private static CryptoStream CreateEncrypted Stream( Stream baseStream, string
password, CryptoStreamMod e streamMode ) {
CryptoStream retValue = null;
SymmetricAlgori thm cryptoAlgorithm = Rijndael.Create ();
PasswordDeriveB ytes pdb = new PasswordDeriveB ytes( password,
new byte[]{
0x12, 0x34, 0x56,
0x78, 0x90,
0xAB,
0xCD, 0xEF,
0x10,
0x11, 0x22,
0x33,
0x44 });
cryptoAlgorithm .Key = pdb.GetBytes( 32 );
cryptoAlgorithm .IV = pdb.GetBytes( 16 );

if( streamMode == CryptoStreamMod e.Read ) {
retValue = new CryptoStream( baseStream,
cryptoAlgorithm .CreateDecrypto r(), streamMode );
}
else {
retValue = new CryptoStream( baseStream,
cryptoAlgorithm .CreateEncrypto r(), streamMode );
}

return retValue;
}

private static void DecryptFile(str ing encryptedFilePa th, string
destinationFile Path, string password) {
FileStream fileStreamSrc = null;
CryptoStream cstream = null;

try {
fileStreamSrc = new FileStream(encr yptedFilePath, FileMode.Open,
FileAccess.Read );
cstream = CreateEncrypted Stream(fileStre amSrc, password,
CryptoStreamMod e.Read);

StreamWriter fsDecrypted = new
StreamWriter(de stinationFilePa th);
fsDecrypted.Wri te(new StreamReader(cs tream).ReadToEn d());
fsDecrypted.Flu sh();
fsDecrypted.Clo se();

cstream.Close() ;
fileStreamSrc.C lose();
}
catch (Exception ex) {
Console.WriteLi ne(ex.ToString( ));
}
}

--------- SAMPLE XML ---------

<States>
<State>
<Name>Illinoi s</Name>
<Population>Unk nown</Population>
<Data>
<Item type="MyType" number="1">
<SubItem utcTime="2004-09-12 17:00:00Z" timeTicks="6323 06052000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-12 17:00:00Z" timeTicks="6323 06052000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-12 20:15:00Z" timeTicks="6323 06169000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-12 20:15:00Z" timeTicks="6323 06169000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-12 20:15:00Z" timeTicks="6323 06169000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-13 00:30:00Z" timeTicks="6323 06322000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-14 01:00:00Z" timeTicks="6323 07204000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:05:00Z" timeTicks="6323 12211000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-20 00:30:00Z" timeTicks="6323 12370000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-21 01:00:00Z" timeTicks="6323 13252000000000"
attr1="False" attr2="False" />
</Item>
<Item type="MyType" number="2">
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 17:00:00Z" timeTicks="6323 12100000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:05:00Z" timeTicks="6323 12211000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-19 20:15:00Z" timeTicks="6323 12217000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-20 00:30:00Z" timeTicks="6323 12370000000000"
attr1="False" attr2="False" />
<SubItem utcTime="2004-09-21 01:00:00Z" timeTicks="6323 13252000000000"
attr1="False" attr2="False" />
</Item>
</Data>
</State>
</States>

[Jon Skeet [C# MVP]]

Tim Barg <Ti*****@discus sions.microsoft .com> wrote:

I am using a CryptoStream to encrypt and then save an xml file. However,
when reading it back in, I am getting a CryptographicEx ception. I am hitting
this because I have some large xml files ( > 10MB) and I only need a certain
node...I use an XmlTextReader to find the node I need, and then attempt to
abort reading in the rest of the xml, but when I close the XmlTextReader I
get the exception. I've included some sample code and xml that should
exhibit the behavior.

Note: If I use Skip() to get to the end of the xml, then I don't get
exceptions, but as I mentioned, the files I'm dealing with can be large and
Skip() takes quite a while to skip over nodes that I don't care about which
really hurts performance. Any ideas?

It would really help if:

a) you could post a short but *complete* program - one we can actually
compile and run, without having to fix up a load of the code

b) you could tell us what the exception message is

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Jon Skeet [C# MVP]]

Tim Barg <Ti*****@discus sions.microsoft .com> wrote:

I am using a CryptoStream to encrypt and then save an xml file. However,
when reading it back in, I am getting a CryptographicEx ception. I am hitting
this because I have some large xml files ( > 10MB) and I only need a certain
node...I use an XmlTextReader to find the node I need, and then attempt to
abort reading in the rest of the xml, but when I close the XmlTextReader I
get the exception. I've included some sample code and xml that should
exhibit the behavior.

Note: If I use Skip() to get to the end of the xml, then I don't get
exceptions, but as I mentioned, the files I'm dealing with can be large and
Skip() takes quite a while to skip over nodes that I don't care about which
really hurts performance. Any ideas?

<snip>

Having looked at this for a few minutes (after fixing up the code you
posted) you can reproduce this without any XML, or even any files. The
problem is that closing a CryptoStream half way through tries to flush
the final buffer, which causes the exception.

It looks like a bug in the framework to me - I suggest you report it to
Microsoft at
http://lab.msdn.microsoft.com/productfeedback/

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Matt]

> It would really help if:

a) you could post a short but *complete* program - one we can actually
compile and run, without having to fix up a load of the code

What exactly did you have to "fix up"? I copied the code into a new console
project and it ran fine without any modification.

[Jon Skeet [C# MVP]]

Matt <Ma**@discussio ns.microsoft.co m> wrote:

It would really help if:

a) you could post a short but *complete* program - one we can actually
compile and run, without having to fix up a load of the code

What exactly did you have to "fix up"? I copied the code into a new console
project and it ran fine without any modification.

Sorry, but you really didn't. Your code doesn't declare a class, to
start with...

In fact, putting the code in a class was all that needed doing - but
that was made fairly unclear due to the formatting of the code. For
instance, the first code block:

private const string MY_PASSWORD = "test"; private const string
SOURCE_XML_FILE _PATH = @"C:\Sample.xml "; private const string
ENCRYPTED_FILE_ PATH = @"C:\Sample.xml e";

would have been rather clearer as:

private const string MY_PASSWORD = "test";
private const string SOURCE_XML_FILE _PATH = @"C:\Sample.xml ";
private const string ENCRYPTED_FILE_ PATH = @"C:\Sample.xml e";

Similarly, at first sight Main (as posted) doesn't appear to have any
closing brace - it's "hidden" at the end of the line.

It's worth considering how the code will appear to someone else when
posted, bearing in mind that that person will never have seen the code
before.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Matt]

> Sorry, but you really didn't. Your code doesn't declare a class, to

start with...

In fact, putting the code in a class was all that needed doing - but
that was made fairly unclear due to the formatting of the code. For
instance, the first code block:

private const string MY_PASSWORD = "test"; private const string
SOURCE_XML_FILE _PATH = @"C:\Sample.xml "; private const string
ENCRYPTED_FILE_ PATH = @"C:\Sample.xml e";

would have been rather clearer as:

private const string MY_PASSWORD = "test";
private const string SOURCE_XML_FILE _PATH = @"C:\Sample.xml ";
private const string ENCRYPTED_FILE_ PATH = @"C:\Sample.xml e";

Similarly, at first sight Main (as posted) doesn't appear to have any
closing brace - it's "hidden" at the end of the line.

It's worth considering how the code will appear to someone else when
posted, bearing in mind that that person will never have seen the code
before.

Ah, you're talking formatting issues. Yes, I agree. We posted the message
through the msdn newsgroup site (http://msdn.microsoft.com/newsgroups) which
appears to have thrown off the line breaks in some weird way.

Also, for reference, we would have preferred to attach a complete program,
but did not see any obvious way to attach files to a post using the msdn site.

Anyway, we suspected it was a bug in the framework (we're using .NET 1.1
Framework). Thanks for taking a look.

Matt

[Jon Skeet [C# MVP]]

Matt <Ma**@discussio ns.microsoft.co m> wrote:

It's worth considering how the code will appear to someone else when
posted, bearing in mind that that person will never have seen the code
before.
Ah, you're talking formatting issues.

Only partly - even without the formatting issues, your code still
didn't declare any classes, going straight from the using directives to
a member declaration.
Yes, I agree. We posted the message
through the msdn newsgroup site (http://msdn.microsoft.com/newsgroups) which
appears to have thrown off the line breaks in some weird way.
It's generally a better idea to use a "real" newsreader IMO, pointing
at news.microsoft. com.
Also, for reference, we would have preferred to attach a complete program,
but did not see any obvious way to attach files to a post using the msdn site.

Anyway, we suspected it was a bug in the framework (we're using .NET 1.1
Framework). Thanks for taking a look.

Yup, it does look like it.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too