Project

General

Profile

Download (2.01 KB)

Bug #198 ยป 0002-This-patch-for-app-layer-ssl-fix-the-bug-198-SSLPa.patch

It's really weird, but it seems that the way of initializing the input was not handled properly by gcc on -O2 and -O3. I changed it to avoid the usage of brackets as array, and directly use a string with hexa data, fixing the length, since it has \0's, an - Pablo Rincon, 06/30/2010 10:12 AM

View differences:

src/app-layer-ssl.c
AppLayerParserResult *output)
{
SCEnter();
SslClient *client = NULL;
SslState *ssl_st = NULL;
/* SSL client message should be larger than 9 bytes as we need to know, to
what is the SSL version and message type */
......
SCReturnInt(1);
}
SslClient *client = (SslClient *)input;
SslState *ssl_st = (SslState *)ssl_state;
client = (SslClient *)input;
ssl_st = (SslState *)ssl_state;
switch (client->msg_type) {
case SSL_CLIENT_HELLO:
......
static int SSLParserTest01(void) {
int result = 0;
Flow f;
uint8_t sslbuf[] = {0x80, 0x31, 0x01, 0x00, 0x02, 0x00, 0x00, 0x00, 0x01 };
uint32_t ssllen = sizeof(sslbuf);
uint8_t *sslbuf = (uint8_t *) "\x80\x31\x01\x00\x02\x00\x00\x00\x01";
/* PrintRawDataFp(stdout, sslbuf, 9); */
uint32_t ssllen = 9;
TcpSession ssn;
memset(&f, 0, sizeof(f));
......
0x2f, 0x34, 0x84, 0x20, 0xc5};
uint32_t ssllen = sizeof(sslbuf);
TcpSession ssn;
AppLayerDetectProtoThreadInit();
memset(&f, 0, sizeof(f));
memset(&ssn, 0, sizeof(ssn));
    (1-1/1)