Doubt in pointer to structure

[karthik537]

static int *p;

void main()
{
    point_update(&p);
    printf("%d",*p);
}

point_update(int **q)
{
   int x = 5;
   *q = &x;
}

In the above pgm, why *p is giving some junk value?

[laserlight]

When control returns from the point_update function, the local variable x no longer exists, hence dereferencing p in main leads to undefined behaviour. Also, to print a pointer, you should be using the %p format specifier, not %d, and you should cast the pointer argument to void*. Oh, and void main should be int main, and you should declare point_update before using it.

[grumpy]

In addition to all that laserlight has said, specify the return type of point_update() as well.

[std10093]

In addition to all that laserlight has said, specify the return type of point_update() as well.

The default return type of a function is int,but it is better to declare it by yourself.
In this case,you should declare it void of course

[qny]

In addition to allthat grumpy and laserlight and std10093 said, you need to have a prototype declared before you call a function that takes a variable number of parameters. In this case, the prototype for the function printf() is in the standard header <stdio.h>

#include <stdio.h>
/* ... */

[jimblumberg]

The default return type of a function is int,but it is better to declare it by yourself.
In this case,you should declare it void of course

Actually that is only true with the outdated C standards, like C89-90. The current standard doesn't allow default return types.

Jim

[std10093]

In addition to allthat grumpy and laserlight and std10093 said, you need to have a prototype declared before you call a function that takes a variable number of parameters. In this case, the prototype for the function printf() is in the standard header <stdio.h>

#include <stdio.h>
/* ... */

Or instead of a prototype,you could define your function before you use it (before main in other words)
definition

int foo(void)
{
    printf("inside foo\n");
}

declaration

int foo(void);

The way that gny is more elegant and it is the one i suggest