AES Spuckt müll aus

  • C#

Es gibt 19 Antworten in diesem Thema. Der letzte Beitrag () ist von ErfinderDesRades.

    AES Spuckt müll aus

    Hey leute,

    Ich habe inzwischen eine sichere Verbindung zwischen Server und Client aufgebaut mittels RSA und AES. Wenn ich jedoch nun z.B. einen Lizenzschlüssel "F4FBF-B60FC-6F3ED-1E5A9" durch den CryptoStream jage(RijndaelManaged), bekomme ich am anderen Ende folgendes wieder heraus: "Ü]Þí£ôÉò×â×$¼ÚD-1E5A9"

    wie man sieht stimmen die letzen 7 Buchstaben wieder überein, doch Alles davor schient nach wie vor verschlüsselt zu sein. Woran kann das liegen?

    ach mist... total vergessen die Funktion drum rum... Danke für dne Hinweis, direkt mal ausgebessert. Die Funktion DoTheThing wird bei nem Buttonklick ausgeführt. der eine Schlüssel steht halt derzeit noch als klartext im Quellcode, wird aber später (da ja auch ma landere Schlüssel abgefragt werden sollen) aus der DB abgefragt.

    Gut dann hier mal dat ganze:
    Server

    C#-Quellcode

    1. using System;
    2. using System.Collections.Generic;
    3. using System.Linq;
    4. using System.Net;
    5. using System.Net.Sockets;
    6. using System.Security.Cryptography;
    7. using System.Text;
    8. using MySql.Data.MySqlClient;
    9. using MySql.Data.Common;
    11. namespace ServerLizenz
    12. {
    13. class Program
    14. {
    15. static void Main(string[] args)
    16. {
    17. Encoding bla = Encoding.GetEncoding("ISO-8859-1");
    19. Int32 port = 14000;
    20. IPAddress localAddr = IPAddress.Parse("192.168.1.177");
    22. TcpListener server = new TcpListener(localAddr, port);
    24. server.Start();
    25. while (true)
    26. {
    27. Console.Write("Waiting for a connection... ");
    29. TcpClient client = server.AcceptTcpClient();
    30. Console.WriteLine("Connected!");
    32. NetworkStream stream = client.GetStream();
    33. stream.ReadTimeout = System.Threading.Timeout.Infinite;
    34. stream.WriteTimeout = System.Threading.Timeout.Infinite;
    36. byte[] KeyBuffer = new Byte[1510];
    37. stream.Read(KeyBuffer, 0, KeyBuffer.Length);
    38. stream.Flush();
    39. RSACryptoServiceProvider RSAC = new RSACryptoServiceProvider(4096);
    40. RSAC.FromXmlString(bla.GetString(KeyBuffer));
    42. RijndaelManaged RM = new RijndaelManaged();
    44. byte[] RMKeyBuffer = RSAC.Encrypt(RM.Key, true);
    45. stream.Write(RMKeyBuffer, 0, RMKeyBuffer.Length);
    46. stream.Flush();
    47. CryptoStream cs = new CryptoStream(stream, RM.CreateDecryptor(), CryptoStreamMode.Read);
    48. byte[] licence = new byte[23];
    49. cs.Read(licence, 0, licence.Length);
    50. cs.Dispose();
    52. string myConnectionString = "SERVER=Server;" +
    53. "DATABASE=Database;" +
    54. "UID=UID;" +
    55. "PASSWORD=Password;";
    57. MySqlConnection mysqlConn = new MySqlConnection(myConnectionString);
    58. MySqlCommand myComm = mysqlConn.CreateCommand();
    59. myComm.CommandText = "SELET * From lizenzen WHERE Lizenznummer='"+bla.GetString(licence)+"'";
    60. mysqlConn.Open();
    61. MySqlDataReader Reader = myComm.ExecuteReader();
    62. string Data = string.Empty;
    63. string blargh = string.Empty;
    64. while (Reader.Read())
    65. {
    67. for (int i = 0; i < Reader.FieldCount; i++)
    68. {
    69. blargh += "|"+Reader.GetDataTypeName(i);
    70. if (Reader.GetDataTypeName(i) == "DATE")
    71. {
    72. try
    73. {
    74. DateTime dt = Reader.GetDateTime(i);
    75. Data += dt.ToShortDateString() +",";
    76. }
    77. catch (Exception)
    78. { Data+= "|-|"; }
    79. }
    80. else
    81. { Data += Reader.GetValue(i).ToString() + ","; }
    82. }
    84. }
    85. CryptoStream csW = new CryptoStream(stream, RM.CreateEncryptor(), CryptoStreamMode.Write);
    86. byte[] DataBuffer= bla.GetBytes(Data);
    87. //stream.Write(DataBuffer, 0, DataBuffer.Length);
    88. csW.Write(DataBuffer, 0, DataBuffer.Length);
    89. csW.Flush();
    90. csW.Close();
    91. csW.Dispose();
    93. stream.Close();
    94. client.Close();
    95. }
    96. Console.ReadKey();
    97. }
    98. }
    99. }


    Client

    C#-Quellcode

    1. using System;
    2. using System;
    3. using System.Collections.Generic;
    4. using System.ComponentModel;
    5. using System.Data;
    6. using System.Drawing;
    7. using System.Net;
    8. using System.Net.Sockets;
    9. using System.Security.Cryptography;
    10. using System.Text;
    11. using System.Windows.Forms;
    13. namespace ClientLizenz
    14. {
    15. public partial class Form1 : Form
    16. {
    17. public Form1()
    18. { InitializeComponent(); }
    20. private void button1_Click(object sender, EventArgs e)
    21. { DoTheThing("F4FBF-B60FC-6F3ED-1E5A9"); }
    23. private void DoTheThing(string Lizenz)
    24. {
    25. //try
    26. //{
    27. textBox2.Text = "Passed Beginning";
    28. Encoding bla = Encoding.GetEncoding("ISO-8859-1");
    29. Int32 port = 14000;
    30. TcpClient client = new TcpClient("192.168.1.177", port);
    31. NetworkStream stream = client.GetStream();
    32. stream.ReadTimeout = System.Threading.Timeout.Infinite;
    33. stream.WriteTimeout = System.Threading.Timeout.Infinite;
    34. textBox2.Text = "Passed Socket Creation";
    36. RSACryptoServiceProvider RSAC = new RSACryptoServiceProvider(4096);
    37. byte[] Key = bla.GetBytes(RSAC.ToXmlString(false));
    38. stream.Write(Key, 0, Key.Length);
    39. textBox2.Text = "Passed sending KEy";
    41. byte[] encryAES = new byte[512];
    42. stream.Read(encryAES, 0, encryAES.Length);
    43. textBox2.Text = "Passed receiving Encrypted AES Key";
    45. byte[] decryAES = RSAC.Decrypt(encryAES, true);
    46. string decryAesString = string.Empty;
    47. textBox2.Text = "Passed Decryption";
    49. foreach (byte item in decryAES)
    50. {
    51. decryAesString += ((int)item).ToString();
    52. }
    53. textBox1.Text = decryAesString;
    55. RijndaelManaged RM = new RijndaelManaged();
    56. RM.Key = decryAES;
    57. CryptoStream csS = new CryptoStream(stream, RM.CreateEncryptor(), CryptoStreamMode.Write);
    58. byte[] Licence = bla.GetBytes(Lizenz);
    59. csS.Write(Licence,0,Licence.Length);
    60. csS.Flush();
    61. csS.Dispose();
    63. CryptoStream csR = new CryptoStream(stream, RM.CreateDecryptor(), CryptoStreamMode.Read);
    64. byte[] Data = new byte[4096];
    65. csR.Read(Data, 0, Data.Length);
    66. csR.Flush();
    67. csR.Dispose();
    69. textBox3.Text = bla.GetString(Data);
    71. //}
    72. //catch (Exception e)
    73. //{
    74. // textBox1.Text = e.ToString();
    75. //}
    76. }
    77. }
    78. }


    Setz bei der SQL geschichte nach der Erstellung des CommandTexts einen Breakpoint, und schau dir mal dessen wert an

    also senden tust du eine key der Länge 755, aber beim Empfang wertest du nicht Stream.Read() aus, die Anzahl der gelesenen Bytes, sondern du verwendest den Keybuffer in voller Länge (1510).
    Dassis schoma komisch - k.A., ob das die Ursache ist.

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „ErfinderDesRades“ ()

    neuer Fehler: Ich habe die RMs (Rijanmanaged) von Server und Client verglichen, und fand unterschiedliche Werte der IV-Property. Der IV muß zusätzlich zum Key übertragen werden (es sei denn, du setzt ihn hardCodet, wassich aber uncool fände.)

    gugge vlt. Verschlüsseln + Autentifizieren

    gut, das mit dem IV habe ich inzwisachen auch herasugefunden... dass ich da nicht früher drauf gekommen bin...

    Auswerten des Streams... aber wie bekomme ich denn die Anzahl der bytes raus, die der Stream liest, bevor ich überhaupt Lese? Dabei brauch ich doch auch direkt mal einen Puffer?
    und ja, der Puffer hat die Doppelte länge, da zuvor noch eine RSA-Schlüssellänge von 4096 gewählt war, was mir aber beim Testen immer viel zu lange dauerte.

    EaranMaleasi schrieb:

    aber wie bekomme ich denn die Anzahl der bytes raus, die der Stream liest

    gugge im Control.Invoke() vs. .BeginInvoke() den Rückgabewert von Stream.Read()

    gugge auch Stream-Konzepte. Das Zusammenstöpseln betreibst du ja bereits, aber die Read-Schleife mittm Puffer.

    Auch brauchst du vmtl. ein weitergehendes Datenübertragungs-Protokoll, da du verschiedenste Daten überträgst: RSA-Key, AES-Key und Daten, und evtl. sogar verschiedenste Arten von Daten.
    Das sollte glaub jeweils unterschieden werden.
    Mein Grundkonzept für son Protokoll sieht vor, dass eine Dateneinheit immer aus Header und Daten besteht. Wobei der Header die Art der Daten und die Daten-Länge angibt.
    Sodass man immer abwechselnd den Header liest, dann die Daten, dann den nächsten Header, dann die Daten etc.
    Da der Header eine feste Länge hat kann jedes übertragene Byte eindeutig zugeordnet werden, und auch klar ist, wann eine Daten-Einheit abgeschlossen ist.

    was derzeit mein Problem ist, dass auf Serverseite, der erste cryptrostream im Read-modus ewig im Read bleibt und nicht die geschriebenen Daten empfängt oder erkennt ,dass Dateien empfangen wurde. schreibe ich jedenfalls auf der Client Seite hinter den ersten Cryptostream ein Dispose oder Close. so endet auf serverseite das Lesen, jedoch mit den exakten Daten...

    Welche Read schleife meinst ud nochmal?

    Keine Sorge, das mit dem Protokoll etc ist nicht vergessen. ich habe sowieso noch einiges zu tun an dem ganzen, es muss auchnoch in einzelnen Threads laufen... Timeouts haben... Obergrenze an Vebrindungen...etc.

    EaranMaleasi schrieb:

    Welche Read schleife meinst ud nochmal?
    guck doch einfach mal in den verlinkten Code, wie da Read() verwendet wird, insbesondere der Rückgabewert, anhand dessen ausgewertet wird, wieviele Bytes bisher übertragen wurden.
    Musste natürlich auf deine Bedingungen anpassen.


    schreibe ich jedenfalls auf der Client Seite hinter den ersten Cryptostream ein Dispose oder Close. so endet auf serverseite das Lesen, jedoch mit den exakten Daten.
    Jo, das ist wohl halt so: Ein CryptoStream muß geschlossen werden, um den Schreibvorgang abzuschließen.
    Pack ihn in ein Using-Block.

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „ErfinderDesRades“ ()

    hmm nun gut, ein Read in eine DoWhile schleife zu packen ist durchaus sinvoll, (auch mal getestet, jeodoch ohne erfolg hinsichtlich meines eigentlichen Problemes), doch bringt mich das kein bisschen weiter mit dem Problem, dass der Crypto Stream nicht mehr mit Lesen aufhören möchte. wenn du mir ein konkretes beispiel hättest, wie man mit dem Cryptostream mehrfach daten hin und her shcickt, wäre ich dir sehr dankbar.

    es geht nicht. Cryptostream (im Grunde AES) verschlüsselt immer den kompletten Text, kann also prinzipiell nicht ein unendlicher Stream sein, im Gegensatz zum Networkstream.
    Aber scheint mir kein Problem, für jeden Lese/Schreib-Vorgang halt einen neuen CryptoStream auf den Networkstream draufzusetzen.

    das meinte ich ja mehr oder minder, weswegen ich ja nicht umsonst den CS mehrfach aufsetze. jedoch ließt auf einer Seiten des Servers der CS unendlich lange, während der client immer schön brav das Lesen beendet. anderst herum, dispose ich nach dem ersten Clientseitigem Schreiben den CryptoStream, erhalte ich die Korrekten Daten auf der Serverseite, kann damit meine DB abfragen durchlaufen lassen und könnte wieder zurückschicken, wäre der NEtworkStream nciht gleich mitvernichtet worden. wasalso kann ich tun um einen CryptoStream zu beenden ohne dabei den darunter liegenden NetworkStream mit ins Nirvana zu reißen?... evtl ein Using?

    Edit: nope ein Using vernichtet den NetworkStream ebenso.

    Eigenen Stream schreiben, der die Daten puffert und ab einer bestimmten Datengröße entschlüsselt und zum lesen/schreiben freigibt. CryptoStream kannst direkt dort integrieren, da du jedesmal einen neuen CryptoStream erstellen müsstest geht es auch über den Umweg msdn.microsoft.com/de-de/libra…phy.icryptotransform.aspx machen. Ohne jedesmal ein neues Objekt erstellen zu müssen.
    Ich wollte auch mal ne total überflüssige Signatur:
    ---Leer---

    so, jetz weiß glaub.
    Problem ist, CryptoStream kannste nicht Flushen. Das liegt an AES, dass da immer ein Block fertig-crypted sein muß, bevor er rausgeht oder reinkommt.

    Also ich hab jetzt im Server einen BinaryReader auf den CS gesetzt, und im Client einen BinaryWriter, und der Client kann beliebig viele Strings an den Server senden, aber der Text kommt erst an, wenn im Client genügend viel hineingeschrieben wurde, dass ein Block rausgeht.
    Und im Client bleibt eiglich immer ein Daten-Rest zurück, sodass auch im Server immer ein Endchen fehlt.

    Aber ansonsten ist das mittm BinaryReader/Writer wirklich nett

    C#-Quellcode

    1. using System;
    2. using System.IO;
    3. using System.Collections.Generic;
    4. using System.Linq;
    5. using System.Net;
    6. using System.Net.Sockets;
    7. using System.Security.Cryptography;
    8. using System.Text;
    9. using MySql.Data.MySqlClient;
    10. namespace A0Console {
    11. static class Program {
    12. static void Main(string[] args) {
    13. Int32 port = 12345;
    14. IPAddress localAddr = IPAddress.Parse("192.168.1.2");
    15. TcpListener server = new TcpListener(localAddr, port);
    16. server.Start();
    17. while(true) {
    18. Console.Write("Waiting for a connection... ");
    19. TcpClient client = server.AcceptTcpClient();
    20. Console.WriteLine("Connected!");
    21. NetworkStream stream = client.GetStream();
    22. byte[] buf = new Byte[1510];
    23. var l = stream.Read(buf, 0, buf.Length);
    24. RSACryptoServiceProvider RSAC = new RSACryptoServiceProvider(4096);
    25. RSAC.ImportCspBlob(buf.Take( 0, l));
    26. RijndaelManaged RM = new RijndaelManaged();
    27. var aesBlob = RM.Key.Concat(RM.IV).ToArray();
    28. byte[] RMKeyBuffer = RSAC.Encrypt(aesBlob, true);
    29. stream.Write(RMKeyBuffer, 0, RMKeyBuffer.Length);
    30. CryptoStream cs = new CryptoStream(stream, RM.CreateDecryptor(), CryptoStreamMode.Read);
    31. var rd = new BinaryReader(cs, Encoding.UTF7);
    32. string s;
    33. do {
    34. s = rd.ReadString();
    35. Console.WriteLine(s);
    36. } while(s!="end");
    37. var i = 9;
    38. }
    39. Console.ReadKey();
    40. }
    41. }
    42. }
    43. //...
    45. public partial class Form1 : Form {
    46. private BinaryWriter _Writer;
    48. public Form1() { InitializeComponent();
    49. var Lizenz = "0815-0815-007-drölf";
    50. Int32 port = 12345;
    51. TcpClient client = new TcpClient("192.168.1.2", port);
    52. var stream = client.GetStream();
    53. RSACryptoServiceProvider RSAC = new RSACryptoServiceProvider(4096);
    54. var cspBlob = RSAC.ExportCspBlob(false);
    55. stream.Write(cspBlob, 0, cspBlob.Length);
    56. byte[] encryAES = new byte[512];
    57. stream.Read(encryAES, 0, encryAES.Length);
    58. byte[] aesBlob = RSAC.Decrypt(encryAES, true);
    59. RijndaelManaged RM = new RijndaelManaged();
    60. RM.Key = aesBlob.Take(0, 32);
    61. RM.IV = aesBlob.Take(32, 16);
    62. CryptoStream csS = new CryptoStream(stream, RM.CreateEncryptor(), CryptoStreamMode.Write);
    63. _Writer = new BinaryWriter(csS, Encoding.UTF7);
    64. _Writer.Write(Lizenz);
    65. }
    67. private void button1_Click(object sender, System.EventArgs e) {
    68. _Writer.Write(textBox1.Text);
    69. }
    70. }