hi, oh ya, one thing i immediately noticed (i didnt look at the code closely) is the common mistake of using strlen() on ciphertext and plaintext... they're not really "text" ! they could have nulls within or no nulls at all so you get unexpected results/crashes/global thermonuclear war...
even "plaintext" is not text (dont use strlen) even though you might be entering a word or sentence... cheers, vishal -- "Thou shalt not follow the null pointer for at it's end madness and chaos lie."
