C# split byte array from file

Question

Hello I'm doing an encryption algorithm which reads bytes from file (any type) and outputs them into a file. The problem is my encryption program takes only blocks of 16 bytes so if the file is bigger it has to be split into blocks of 16, or if there's a way to read 16 bytes from the file each time it's fine.

The algorithm is working fine with hard coded input of 16 bytes. The ciphered result has to be saved in a list or array because it has to be deciphered the same way later. I can't post all my program but here's what I do in main so far and cannot get results

static void Main(String[] args)
{
    byte[] bytes = File.ReadAllBytes("path to file");
    var stream = new StreamReader(new MemoryStream(bytes));
    byte[] cipherText = new byte[16];
    byte[] decipheredText = new byte[16];

    Console.WriteLine("\nThe message is: ");
    Console.WriteLine(stream.ReadToEnd());

    AES a = new AES(keyInput);
    var list1 = new List<byte[]>();
    for (int i = 0; i < bytes.Length; i+=16)
    {
        a.Cipher(bytes, cipherText);
        list1.Add(cipherText);
    }

    Console.WriteLine("\nThe resulting ciphertext is: ");
    foreach (byte[] b in list1)
    {       
        ToBytes(b);
    }
}

I know that my loops always add the first 16 bytes from the byte array but I tried many ways and nothing work. It won't let me index the bytes array or copy an item to a temp variable like temp = bytes[i]. The ToBytes method is irrelevant, it just prints the elements as bytes.

Sergey Vyacheslavovich Brunov · Accepted Answer · 2013-01-25 20:10:21Z

1

I would like to recommend you to change the interface for your Cipher() method: instead of passing the entire array, it would be better to pass the source and destination arrays and offset - block by block encryption.

Pseudo-code is below.

void Cipher(byte[] source, int srcOffset, byte[] dest, int destOffset)
{
    // Cipher these bytes from (source + offset) to (source + offset + 16),
    // write the cipher to (dest + offset) to (dest + offset + 16)
    // Also I'd recommend to check that the source and dest Length is less equal to (offset + 16)!
}

Usage:

For small files (one memory allocation for destination buffer, block by block encryption):

// You can allocate the entire destination buffer before encryption!
byte[] sourceBuffer = File.ReadAllBytes("path to file");
byte[] destBuffer = new byte[sourceBuffer.Length];

// Encrypt each block.
for (int offset = 0; i < sourceBuffer.Length; offset += 16)
{
    Cipher(sourceBuffer, offset, destBuffer, offset);
}

So, the main advantage of this approach - it elimitates additional memory allocations: the destination array is allocated at once. There is also no copy-memory operations.

For files of any size (streams, block by block encryption):

    byte[] inputBlock = new byte[16];
    byte[] outputBlock = new byte[16];
    using (var inputStream = File.OpenRead("input path"))
    using (var outputStream = File.Create("output path"))
    {
        int bytesRead;
        while ((bytesRead = inputStream.Read(inputBlock, 0, inputBlock.Length)) > 0)
        {
            if (bytesRead < 16)
            {
                // Throw or use padding technique.
                throw new InvalidOperationException("Read block size is not equal to 16 bytes");

                // Fill the remaining bytes of input block with some bytes.
                // This operation for last block is called "padding".
                // See http://en.wikipedia.org/wiki/Block_cipher_modes_of_operation#Padding
            }

            Cipher(inputBlock, 0, outputBlock, 0);
            outputStream.Write(outputBlock, 0, outputBlock.Length);
        }
    }

edited Jan 25, 2013 at 20:10

answered Jan 25, 2013 at 19:11

Sergey Vyacheslavovich Brunov

18.2k7 gold badges53 silver badges85 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

geolykos Over a year ago

Didn't think of getting the size of the file before I start processing it. Thanks! I'll try that too

Sergey Vyacheslavovich Brunov Over a year ago

@geolykos, please see the streaming approach.

geolykos Over a year ago

Nice. One problem I have though is that I need all the bytes ciphered. Even if they are not multiplicative of 16. I'm getting the last part by modulo with 16.

JerKimball · Accepted Answer · 2013-01-25 19:10:15Z

1

No need to read the whole mess into memory if you can only process it a bit at a time...

var filename = @"c:\temp\foo.bin";
using(var fileStream = new FileStream(filename, FileMode.Open))
{
    var buffer = new byte[16];
    var bytesRead = 0;
    while((bytesRead = fileStream.Read(buffer, 0, buffer.Length)) > 0)
    {
        // do whatever you need to with the next 16-byte block
        Console.WriteLine("Read {0} bytes: {1}", 
                bytesRead, 
                string.Join(",", buffer));
    }
}

answered Jan 25, 2013 at 19:10

JerKimball

17k3 gold badges45 silver badges57 bronze badges

Comments

itsme86 · Accepted Answer · 2013-01-25 19:04:56Z

0

You can use Array.Copy

byte[] temp = new byte[16];
Array.Copy(bytes, i, temp, 0, 16);

answered Jan 25, 2013 at 19:04

itsme86

19.6k4 gold badges44 silver badges60 bronze badges

1 Comment

geolykos Over a year ago

It's my first program in C# so it's kind of confusing but yeah that's a very nice and simple way to try. Thanks!

Collectives™ on Stack Overflow

C# split byte array from file

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related