сравнение ХЭШЕЙ? , как сравнить два хэша возвращённых из функции

[woojin]

всем привет!
пытаюсь написать небольшую програмку под FreeBSD 10.x

есть функция создающая MD5 хэш

char* MD5 (const char* context)

она возвращает (без терминатора "\0")

char buf[32];
return buf; // хотя тут ругается "warning: address of stack memory associated with local variable 'buf' returned"

подскажите как сравнить две такие строки, проверить их идентичность

делаю так

char bufr1[]="123654";
char bufr2[]="123654";
 
char* ch1 = MD5((const char*)bufr1);
char* ch2 = MD5((const char*)bufr2);
 
int sz1=sizeof(ch1);
int sz2=sizeof(ch2);
 
int cmp = strncmp(ch1, ch2, sz2);

но результат как мне кажется получается не правильный, т.к. sizeof возвращает размер(4) только первого символа

как правильно вернуть массив или как сравнить данные из полученных адресов?

P.S. в C/C++ разбираюсь на столько поверхностно, что можно сказать лечу, а не парю, над поверхностью
и извините если такого рода тема уже была, 5-й час по форуму ползаю... не нашёл

[Олег М]

Цитата woojin @ 16.03.17, 08:57

она возвращает (без терминатора "\0")

char buf[32];
return buf; // хотя тут ругается "warning: address of stack memory associated with local variable 'buf' returned"

Правильно ругается ты возвращаешь адрес буфера, который удаляется стазу после return;

Добавлено 16.03.17, 09:02

Цитата woojin @ 16.03.17, 08:57

как правильно вернуть массив или как сравнить данные из полученных адресов?

Лучше передавать адрес и размер буфера в качестве параметров

Добавлено 16.03.17, 09:04
Что-то типа
void MD5 (const char* context, char *buf, size_t sz)

Добавлено 16.03.17, 09:08

Цитата woojin @ 16.03.17, 08:57

int sz1=sizeof(ch1);
int sz2=sizeof(ch2);

Эти выражения возвращают sizof(char *) == 4 или 8

Добавлено 16.03.17, 09:11
Надо как-то так

char bufr1[]="123654";
char bufr2[]="123654";
 
static const size_t _sz = 32;
char hash1[_sz] = {0};
char hash2[_sz] = {0};
 
MD5(bufr1, hash1);
MD5(bufr2, hash2);
 
int cmp = memcmp(hash1, hash1, _sz) ;

[shm]

Цитата woojin @ 16.03.17, 08:57

т.к. sizeof возвращает размер(4) только первого символа

Размер указателя она в твоем случае возвращает.

[Олег М]

Цитата woojin @ 16.03.17, 08:57

nt cmp = strncmp(ch1, ch2, sz2);

Кстати, здесь наверное лучше использовать memcmp

[woojin]

Цитата Олег М @ 16.03.17, 09:00

Эти выражения возвращают sizof(char *) == 4 или 8

Цитата woojin @ 16.03.17, 08:57

sizeof возвращает размер(4)

а где тут

Цитата Олег М @ 16.03.17, 09:00

MD5(bufr1, hash1);
MD5(bufr2, hash2);

третий параметр?

так получается лучше в функцию отправлять ссылку на уже созданный буфер возврата?
по этой ссылке в функции что то делать и по её завершении сравнивать через memcmp?

а настоящего возврата массива символов из функции (массив символов = функция(параметр))не получится сделать - мне кажется просто это красивее

в общем функции для работы со строками (массивами символов) надо писать сразу с тремя параметрами:

void primer(const char* in, char* out, size_t SizeOfOut)

типа такого?

а если надо функцию размер возврата из которой будет не известен, что делать в этом случае (на примере данной MD5)?

Сообщение отредактировано: woojin - 16.03.17, 10:04

[Олег М]

Цитата woojin @ 16.03.17, 10:01

третий параметр?

Это по-настроению. В твоём случае MD5 всегда возвращает массив определённого размера

Добавлено 16.03.17, 10:04

Цитата woojin @ 16.03.17, 10:01

так получается лучше в функцию отправлять ссылку на уже созданный буфер возврата?

Это не буфер возврата. Это массив, куда будут записаны возвращаемые данные

Добавлено 16.03.17, 10:05

Цитата woojin @ 16.03.17, 10:01

по этой ссылке в функции что то делать и по её завершении сравнивать через memcmp?

В функции заполняешь этот массив и всё

[woojin]

Цитата Олег М @ 16.03.17, 10:03

Это не буфер возврата. Это массив, куда будут записаны возвращаемые данные

согласен, не правильно сформулировал, ссылка на массив

Сообщение отредактировано: woojin - 16.03.17, 10:06

[shm]

Цитата woojin @ 16.03.17, 10:01

а если надо функцию размер возврата из которой будет не известен, что делать в этом случае (на примере данной MD5)?

std::string

[Олег М]

Цитата woojin @ 16.03.17, 10:01

а настоящего возврата массива символов из функции (массив символов = функция(параметр))не получится сделать - мне кажется просто это красивее

Нет, не будет. В этом случае тебе нужно будет выделять память динамически и потом как-то удалять её. На стеке, в данном случае, быстрее и лучше

Добавлено 16.03.17, 10:10

Цитата woojin @ 16.03.17, 10:01

void primer(const char* in, char* out, size_t SizeOfOut)

типа такого?

а если надо функцию размер возврата из которой будет не известен, что делать в этом случае (на примере данной MD5)?
Сообщение отредактировано: woojin - 2 минуты назад

Когда неизвестен, то либо делаешь так и проверяешь внутри размер. Либо с динамическим выделением, что-то типа

std::pair<std::unique_ptr<char[]>, size_t> primer(const char* in)

Не думаю, что тебе это нужно

[woojin]

Цитата Олег М @ 16.03.17, 10:06

Не думаю, что тебе это нужно

согласен - сильно замудрено, я не настолько силён

Добавлено 16.03.17, 11:57
сделал проверку

        static const size_t _sz = 32;
        char ch1[_sz] = {'\0'};
        char ch2[_sz] = {'\0'};
        char buff1[] = "123654";
        char buff2[] = "123456";
        MD5(buff1, ch1, _sz); // в отладчике "d41d8cd98f00b204e9800998ecf8427ed41d8cd98f00b204e9800998ecf8427e"
        MD5(buff2, ch2, _sz); // в отладчике "d41d8cd98f00b204e9800998ecf8427e"
        int cmp1 = strncmp(ch1, ch2, _sz);
        int cmp2 = memcmp(ch1, ch2, _sz);

и cmp1 и cmp оба вернули 0 - почему?
получается что сравнение не произошло вообще (от слова совсем )

Сообщение отредактировано: woojin - 16.03.17, 11:58

[Олег М]

Цитата woojin @ 16.03.17, 11:26

и cmp1 и cmp оба вернули 0 - почему?

Потому что ты сравниваешь первые 32 байта, а они вроде одинаковые

Добавлено 16.03.17, 12:01
Покажи, как реализована функция MD5

Сообщение отредактировано: Олег М - 16.03.17, 12:01

[shm]

Еще один забаненный в гугле?
http://www.cplusplus.com/reference/cstring/strncmp/

Цитата

Returns an integral value indicating the relationship between the strings:
return value indicates
<0 the first character that does not match has a lower value in str1 than in str2
0 the contents of both strings are equal
>0 the first character that does not match has a greater value in str1 than in str2

[Олег М]

Цитата woojin @ 16.03.17, 11:26

MD5(buff1, ch1, _sz); // в отладчике "d41d8cd98f00b204e9800998ecf8427ed41d8cd98f00b204e9800998ecf8427e"

Кстати, как ты умудрился в 32-байтный массив записать строку, длиной 64 байта?

[shm]

Вангую, что перепутаны местами параметры функции MD5. Попробуй const char buff1, заорет ли компилятор.

[woojin]

Цитата shm @ 16.03.17, 12:03

Еще один забаненный в гугле?

читал, ну вот и получается что они одинаковые

Цитата Олег М @ 16.03.17, 12:11

Кстати, как ты умудрился в 32-байтный массив записать строку, длиной 64 байта?

хороший вопрос - ответ наверное знает NetBeans и gdb

Цитата Олег М @ 16.03.17, 12:01

Покажи, как реализована функция MD5

md5c.h

#ifndef _MD5C_H_
#define _MD5C_H_ 1
 
#if defined(__MINGW32__)
#   include <basetyps.h>
#endif
 
#if !defined(_WIN32)
#include <sys/types.h>
 
#ifdef sun
typedef uint32_t uint32;
#else
typedef u_int32_t uint32;
#endif /* sun */
 
#else
typedef unsigned __int32 uint32;
#endif /* _WIN32 */
 
#ifndef POINTER_TYPE
#define POINTER_TYPE    1
typedef unsigned char *POINTER;
#endif
 
#ifdef __cplusplus
extern "C" {
#endif
 
/* MD5 context. */
typedef struct {
  uint32 state[4];                                   /* state (ABCD) */
  uint32 count[2];        /* number of bits, modulo 2^64 (lsb first) */
  unsigned char buffer[64];                         /* input buffer */
} MD5_CTX;
 
void MD5_Init (MD5_CTX *);
void MD5_Update (MD5_CTX *, const unsigned char *, unsigned int);
void MD5_Final (unsigned char [16], MD5_CTX *);
 
#ifdef __cplusplus
}
#endif
 
#endif

md5c.c

#include "md5c.h"
 
/* Constants for MD5Transform routine.
 */
#define S11 7
#define S12 12
#define S13 17
#define S14 22
#define S21 5
#define S22 9
#define S23 14
#define S24 20
#define S31 4
#define S32 11
#define S33 16
#define S34 23
#define S41 6
#define S42 10
#define S43 15
#define S44 21
 
static void MD5Transform (uint32 [4], const unsigned char [64]);
static void Encode (unsigned char *, uint32 *, unsigned int);
static void Decode (uint32 *, const unsigned char *, unsigned int);
static void MD5_memcpy (POINTER, POINTER, unsigned int);
static void MD5_memset (POINTER, int, unsigned int);
 
static const unsigned char PADDING[64] = {
  0x80, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
  0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0
};
 
/* F, G, H and I are basic MD5 functions.
 */
#define F(x, y, z) (((x) & (y)) | ((~x) & (z)))
#define G(x, y, z) (((x) & (z)) | ((y) & (~z)))
#define H(x, y, z) ((x) ^ (y) ^ (z))
#define I(x, y, z) ((y) ^ ((x) | (~z)))
 
/* ROTATE_LEFT rotates x left n bits.
 */
#define ROTATE_LEFT(x, n) (((x) << (n)) | ((x) >> (32-(n))))
 
/* FF, GG, HH, and II transformations for rounds 1, 2, 3, and 4.
   Rotation is separate from addition to prevent recomputation.
 */
#define FF(a, b, c, d, x, s, ac) { \
    (a) += F ((b), (c), (d)) + (x) + (uint32)(ac); \
    (a) = ROTATE_LEFT ((a), (s)); \
    (a) += (b); \
  }
#define GG(a, b, c, d, x, s, ac) { \
    (a) += G ((b), (c), (d)) + (x) + (uint32)(ac); \
    (a) = ROTATE_LEFT ((a), (s)); \
    (a) += (b); \
  }
#define HH(a, b, c, d, x, s, ac) { \
    (a) += H ((b), (c), (d)) + (x) + (uint32)(ac); \
    (a) = ROTATE_LEFT ((a), (s)); \
    (a) += (b); \
  }
#define II(a, b, c, d, x, s, ac) { \
    (a) += I ((b), (c), (d)) + (x) + (uint32)(ac); \
    (a) = ROTATE_LEFT ((a), (s)); \
    (a) += (b); \
  }
 
/* MD5 initialization. Begins an MD5 operation, writing a new context.
 */
void MD5_Init (MD5_CTX *context)                                        /* context */
{
  context->count[0] = context->count[1] = 0;
 
  /* Load magic initialization constants.
   */
  context->state[0] = 0x67452301;
  context->state[1] = 0xefcdab89;
  context->state[2] = 0x98badcfe;
  context->state[3] = 0x10325476;
}
 
/* MD5 block update operation. Continues an MD5 message-digest
     operation, processing another message block, and updating the
     context.
 */
void MD5_Update (MD5_CTX *context,          /* context */
        const unsigned char *input,         /* input block */
        unsigned int inputLen)              /* length of input block */
{
  unsigned int i, index, partLen;
 
  /* Compute number of bytes mod 64 */
  index = (unsigned int)((context->count[0] >> 3) & 0x3F);
 
  /* Update number of bits */
  if ((context->count[0] += ((uint32)inputLen << 3))
      < ((uint32)inputLen << 3))
    context->count[1]++;
  context->count[1] += ((uint32)inputLen >> 29);
 
  partLen = 64 - index;
 
  /* Transform as many times as possible.
   */
  if (inputLen >= partLen) {
    MD5_memcpy
      ((POINTER)&context->buffer[index], (POINTER)input, partLen);
    MD5Transform (context->state, context->buffer);
 
    for (i = partLen; i + 63 < inputLen; i += 64)
      MD5Transform (context->state, &input[i]);
 
    index = 0;
  }
  else
    i = 0;
 
  /* Buffer remaining input */
  MD5_memcpy
    ((POINTER)&context->buffer[index], (POINTER)&input[i],
     inputLen-i);
}
 
/* MD5 finalization. Ends an MD5 message-digest operation, writing the
     the message digest and zeroizing the context.
 */
void MD5_Final (unsigned char digest[16],               /* message digest */
        MD5_CTX *context)                              /* context */
{
  unsigned char bits[8];
  unsigned int index, padLen;
 
  /* Save number of bits */
  Encode (bits, context->count, 8);
 
  /* Pad out to 56 mod 64.
   */
  index = (unsigned int)((context->count[0] >> 3) & 0x3f);
  padLen = (index < 56) ? (56 - index) : (120 - index);
  MD5_Update (context, PADDING, padLen);
 
  /* Append length (before padding) */
  MD5_Update (context, bits, 8);
 
  /* Store state in digest */
  Encode (digest, context->state, 16);
 
  /* Zeroize sensitive information.
   */
  MD5_memset ((POINTER)context, 0, sizeof (*context));
}
 
/* MD5 basic transformation. Transforms state based on block.
 */
static void MD5Transform (uint32 state[4], const unsigned char block[64])
{
  uint32 a = state[0], b = state[1], c = state[2], d = state[3], x[16];
 
  Decode (x, block, 64);
 
  /* Round 1 */
  FF (a, b, c, d, x[ 0], S11, 0xd76aa478); /* 1 */
  FF (d, a, b, c, x[ 1], S12, 0xe8c7b756); /* 2 */
  FF (c, d, a, b, x[ 2], S13, 0x242070db); /* 3 */
  FF (b, c, d, a, x[ 3], S14, 0xc1bdceee); /* 4 */
  FF (a, b, c, d, x[ 4], S11, 0xf57c0faf); /* 5 */
  FF (d, a, b, c, x[ 5], S12, 0x4787c62a); /* 6 */
  FF (c, d, a, b, x[ 6], S13, 0xa8304613); /* 7 */
  FF (b, c, d, a, x[ 7], S14, 0xfd469501); /* 8 */
  FF (a, b, c, d, x[ 8], S11, 0x698098d8); /* 9 */
  FF (d, a, b, c, x[ 9], S12, 0x8b44f7af); /* 10 */
  FF (c, d, a, b, x[10], S13, 0xffff5bb1); /* 11 */
  FF (b, c, d, a, x[11], S14, 0x895cd7be); /* 12 */
  FF (a, b, c, d, x[12], S11, 0x6b901122); /* 13 */
  FF (d, a, b, c, x[13], S12, 0xfd987193); /* 14 */
  FF (c, d, a, b, x[14], S13, 0xa679438e); /* 15 */
  FF (b, c, d, a, x[15], S14, 0x49b40821); /* 16 */
 
  /* Round 2 */
  GG (a, b, c, d, x[ 1], S21, 0xf61e2562); /* 17 */
  GG (d, a, b, c, x[ 6], S22, 0xc040b340); /* 18 */
  GG (c, d, a, b, x[11], S23, 0x265e5a51); /* 19 */
  GG (b, c, d, a, x[ 0], S24, 0xe9b6c7aa); /* 20 */
  GG (a, b, c, d, x[ 5], S21, 0xd62f105d); /* 21 */
  GG (d, a, b, c, x[10], S22,  0x2441453); /* 22 */
  GG (c, d, a, b, x[15], S23, 0xd8a1e681); /* 23 */
  GG (b, c, d, a, x[ 4], S24, 0xe7d3fbc8); /* 24 */
  GG (a, b, c, d, x[ 9], S21, 0x21e1cde6); /* 25 */
  GG (d, a, b, c, x[14], S22, 0xc33707d6); /* 26 */
  GG (c, d, a, b, x[ 3], S23, 0xf4d50d87); /* 27 */
  GG (b, c, d, a, x[ 8], S24, 0x455a14ed); /* 28 */
  GG (a, b, c, d, x[13], S21, 0xa9e3e905); /* 29 */
  GG (d, a, b, c, x[ 2], S22, 0xfcefa3f8); /* 30 */
  GG (c, d, a, b, x[ 7], S23, 0x676f02d9); /* 31 */
  GG (b, c, d, a, x[12], S24, 0x8d2a4c8a); /* 32 */
 
  /* Round 3 */
  HH (a, b, c, d, x[ 5], S31, 0xfffa3942); /* 33 */
  HH (d, a, b, c, x[ 8], S32, 0x8771f681); /* 34 */
  HH (c, d, a, b, x[11], S33, 0x6d9d6122); /* 35 */
  HH (b, c, d, a, x[14], S34, 0xfde5380c); /* 36 */
  HH (a, b, c, d, x[ 1], S31, 0xa4beea44); /* 37 */
  HH (d, a, b, c, x[ 4], S32, 0x4bdecfa9); /* 38 */
  HH (c, d, a, b, x[ 7], S33, 0xf6bb4b60); /* 39 */
  HH (b, c, d, a, x[10], S34, 0xbebfbc70); /* 40 */
  HH (a, b, c, d, x[13], S31, 0x289b7ec6); /* 41 */
  HH (d, a, b, c, x[ 0], S32, 0xeaa127fa); /* 42 */
  HH (c, d, a, b, x[ 3], S33, 0xd4ef3085); /* 43 */
  HH (b, c, d, a, x[ 6], S34,  0x4881d05); /* 44 */
  HH (a, b, c, d, x[ 9], S31, 0xd9d4d039); /* 45 */
  HH (d, a, b, c, x[12], S32, 0xe6db99e5); /* 46 */
  HH (c, d, a, b, x[15], S33, 0x1fa27cf8); /* 47 */
  HH (b, c, d, a, x[ 2], S34, 0xc4ac5665); /* 48 */
 
  /* Round 4 */
  II (a, b, c, d, x[ 0], S41, 0xf4292244); /* 49 */
  II (d, a, b, c, x[ 7], S42, 0x432aff97); /* 50 */
  II (c, d, a, b, x[14], S43, 0xab9423a7); /* 51 */
  II (b, c, d, a, x[ 5], S44, 0xfc93a039); /* 52 */
  II (a, b, c, d, x[12], S41, 0x655b59c3); /* 53 */
  II (d, a, b, c, x[ 3], S42, 0x8f0ccc92); /* 54 */
  II (c, d, a, b, x[10], S43, 0xffeff47d); /* 55 */
  II (b, c, d, a, x[ 1], S44, 0x85845dd1); /* 56 */
  II (a, b, c, d, x[ 8], S41, 0x6fa87e4f); /* 57 */
  II (d, a, b, c, x[15], S42, 0xfe2ce6e0); /* 58 */
  II (c, d, a, b, x[ 6], S43, 0xa3014314); /* 59 */
  II (b, c, d, a, x[13], S44, 0x4e0811a1); /* 60 */
  II (a, b, c, d, x[ 4], S41, 0xf7537e82); /* 61 */
  II (d, a, b, c, x[11], S42, 0xbd3af235); /* 62 */
  II (c, d, a, b, x[ 2], S43, 0x2ad7d2bb); /* 63 */
  II (b, c, d, a, x[ 9], S44, 0xeb86d391); /* 64 */
 
  state[0] += a;
  state[1] += b;
  state[2] += c;
  state[3] += d;
 
  /* Zeroize sensitive information.
   */
  MD5_memset ((POINTER)x, 0, sizeof (x));
}
 
/* Encodes input (uint32) into output (unsigned char). Assumes len is
     a multiple of 4.
 */
static void Encode (unsigned char *output, uint32 *input, unsigned int len)
{
  unsigned int i, j;
 
  for (i = 0, j = 0; j < len; i++, j += 4) {
    output[j] = (unsigned char)(input[i] & 0xff);
    output[j+1] = (unsigned char)((input[i] >> 8) & 0xff);
    output[j+2] = (unsigned char)((input[i] >> 16) & 0xff);
    output[j+3] = (unsigned char)((input[i] >> 24) & 0xff);
  }
}
 
/* Decodes input (unsigned char) into output (uint32). Assumes len is
     a multiple of 4.
 */
static void Decode (uint32 *output, const unsigned char *input, unsigned int len)
{
  unsigned int i, j;
 
  for (i = 0, j = 0; j < len; i++, j += 4)
    output[i] = ((uint32)input[j]) | (((uint32)input[j+1]) << 8) |
      (((uint32)input[j+2]) << 16) | (((uint32)input[j+3]) << 24);
}
 
/* Note: Replace "for loop" with standard memcpy if possible.
 */
static void MD5_memcpy (POINTER output, POINTER input, unsigned int len)
{
  unsigned int i;
 
  for (i = 0; i < len; i++)
    output[i] = input[i];
}
 
/* Note: Replace "for loop" with standard memset if possible.
 */
static void MD5_memset (POINTER output, int value, unsigned int len)
{
  unsigned int i;
 
  for (i = 0; i < len; i++)
    ((char *)output)[i] = (char)value;
}

сделал обёртку
md5.hpp

#ifndef MD5_HPP
#define MD5_HPP
 
#include <string.h>
 
void MD5 (const char* context, char* out, size_t size_out);
 
#endif /* MD5_HPP */

md5.cpp

#include "md5.hpp"
#include "md5c.h"
 
void MD5 (const char* context, char* out, size_t size_out) {
    MD5_CTX ctx;
    MD5_Init(&ctx);
    size_t vt = 0;
    
    MD5_Update(&ctx, (unsigned char*) context, vt);
 
    unsigned char tmp[16];
    MD5_Final(tmp, &ctx);
    
 
    static const char hex[] = "0123456789abcdef";
    
    for (int i = 0, j = 0; i<sizeof (tmp); i++) {
        out[j++] = hex[(tmp[i] >> 4)&0x0f];
        out[j++] = hex[tmp[i]&0x0f];
    }
}

Добавлено 16.03.17, 12:24
до моей обёртки
использовалась функция в другом исходнике так:

    const char* s=lua_tolstring(L,1,&l);
 
    if(!s)
        lua_pushnil(L);
    else
    {
        MD5_CTX ctx;
        MD5_Init(&ctx);            
        MD5_Update(&ctx,(unsigned char*)s,l);
 
        unsigned char tmp[16];
        MD5_Final(tmp,&ctx);
 
        char buf[sizeof(tmp)*2];
 
        static const char hex[]="0123456789abcdef";
        for(int i=0,j=0;i<sizeof(tmp);i++)
        {
            buf[j++]=hex[(tmp[i]>>4)&0x0f];
            buf[j++]=hex[tmp[i]&0x0f];
        }
    }

в нём работало

Добавлено 16.03.17, 12:25

Цитата shm @ 16.03.17, 12:13

Вангую, что перепутаны местами параметры функции MD5. Попробуй const char buff1, заорет ли компилятор.

вроде всё правильно сделал - нет?