CODEWITHSUNDEEP
cmmap
user3693586
user3693586

Reputation: 1287

mmap return value during error

I could see in the mmap man page the return value during error condition is (void *)-1.

how c compiler will treat (void *) before a constant, here -1.

is the following code snippet is the correct way of checking the error value of mmap?

int *p;
p = (int *)mmap();
if(p == -1)
        printf("error \n");

do we need to use the following error condition check.

if(*p == -1)
        printf("error \n");

Upvotes: 4

Views: 2693

Answers (1)

Mohit Jain
Mohit Jain

Reputation: 30489

int *p = NULL;
void *ret = mmap();
if(ret == MAP_FAILED) {  /* Or if(ret == (void *)-1) */
  /* error */
} else {
  p = ret;
}

Check literally with (void *)-1 or use MAP_FAILED macro which is defined to (void *)-1.

if(*p == -1) is incorrect as you can not dereference the pointer unless you ensure the validity of the pointer. Otherwise the behaviour of the program is undefined.

Upvotes: 6

Related Questions