std::ofstream prints extra bytes when given an array of unsigned char, but only in Release mode

Question

I'm writing a C++ class to add a digital signature to a PDF, using OpenSSL to generate a SHA1 hash, which I then print to a file:

unsigned char hash[SHA_DIGEST_LENGTH];
SHA1((unsigned char *)temp.c_str(), temp.size(), hash);

std::ofstream fout("resources/hash.out", std::ios::binary);
fout << hash;
fout.close();

This code works as expected in Debug mode. In Release mode, fout << hash prints 30 instead of SHA_DIGEST_LENGTH = 20 bytes, the last 10 of which look like garbage (maybe buffer overflow?)

My current workaround is to print each character instead of streaming:

for (int i=0; i<SHA_DIGEST_LENGTH; ++i)
    fout.put(hash[i]);

Which works in both build modes, but I'm curious what could cause the stream operator to misread the length of the string. Does anybody have any thoughts?

For the record, I'm compiling with MSVC++ 12 (x86_amd64).

Also note temp.size() does not include the null terminator. — NathanOliver
– NathanOliver, Commented Aug 8, 2016 at 14:54

Deduplicator · Accepted Answer · 2016-08-08 15:03:02Z

3

There is a stream-inserter for NUL-terminated strings, which is the one selected.

But you are trying to output an array of fixed length containing arbitrary bytes, which is something completely different, leading to UB.

Just use memberfunction write(buffer, size) instead, which is designed for that.

answered Aug 8, 2016 at 15:03

Deduplicator

46k7 gold badges73 silver badges125 bronze badges

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

Comments

Baum mit Augen · Accepted Answer · 2016-08-08 15:03:08Z

2

You need to take into account that OpenSSL's SHA methods don't null-terminate the output. The normal operator << requires null-termination, so you cannot use it directly.

To print this "unterminated" string with C++ streams, see e.g.

C++ non null terminated char array outputting

for printf e.g.

Using printf with a non-null terminated string

answered Aug 8, 2016 at 15:03

Baum mit Augen♦

50.4k25 gold badges153 silver badges187 bronze badges

Collectives™ on Stack Overflow

std::ofstream prints extra bytes when given an array of unsigned char, but only in Release mode

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related