memset & alignment

[dragoncoder]

Please have a look at the following code snippet.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void* func ( int i )
{
void* ptr = malloc(sizeof(int));
memset(ptr, i, sizeof(int));
return ptr;
}

int main()
{
int i = 123;
int res = *((int*)func(i));
printf("i is %d\n", res);
return 0;
}

[l34@premier rnd]$ gcc p1.c
[l34@premier rnd]$ ./a.out
i is 2071690107

I don't understand what the problrm is, as I was expecting 123 to get
printed. Can somebody tell why this garbage is being printed.

Thanks.

[pete]

dragoncoder wrote:

memset(ptr, i, sizeof(int));

memcpy(ptr, &i, sizeof i);

--
pete

[Chris Dollin]

dragoncoder wrote:

Please have a look at the following code snippet.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void* func ( int i )
{
void* ptr = malloc(sizeof(int));
memset(ptr, i, sizeof(int));

You want to set each byte of the int-sized object at *ptr to
i? OK. Seems a bit strange, but you know best.

return ptr;
}

int main()
{
int i = 123;
int res = *((int*)func(i));
printf("i is %d\n", res);
return 0;
}

[l34@premier rnd]$ gcc p1.c
[l34@premier rnd]$ ./a.out
i is 2071690107

I don't understand what the problrm is, as I was expecting 123 to get
printed.

Bad expectation.

If `sizeof int` is 4 and there are 8 bits-per-byte (likely guesses), you've
set `res` to

(123 << 24) | (123 << 16) | (123 << 8) | 123

which by an amazing coincidence prints as 2071690107.

Surely surely surely you would have been better with

int *func( int i )
{
int *ptr = malloc( sizeof (*ptr) );
*ptr = i;
return ptr;
}
??

--
Chris "seeker" Dollin
"Who are you? What do you want?" /Babylon 5/

[Neroku]

dragoncoder ha escrito:

Please have a look at the following code snippet.

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void* func ( int i )
{
void* ptr = malloc(sizeof(int));
memset(ptr, i, sizeof(int));
return ptr;
}

int main()
{
int i = 123;
int res = *((int*)func(i));
printf("i is %d\n", res);
return 0;
}

[l34@premier rnd]$ gcc p1.c
[l34@premier rnd]$ ./a.out
i is 2071690107

I don't understand what the problrm is, as I was expecting 123 to get
printed. Can somebody tell why this garbage is being printed.

Thanks.

memset(ptr, i, n) sets each byte of n bytes pointed by ptr to i.
If n is 4, what you get pointed by ptr is:

01111011 01111011 01111011 01111011

123(decimal) = 01111011(binary)

which is 2071690107 in decimal.