Changing pointer address - function

Question

I have problem with pointers. This is working fine -

int main(void){
    char *w;
    w = calloc(20, sizeof(char));
    w = "ab";

    printf("%c",*w);
    w = w + sizeof(char);
    printf("%c",*w);

    return 0;
}

but if i use function like:

void por(char *t){
    t = t + sizeof(char);
}

and

int main(void){
    char *w;
    w = calloc(20, sizeof(char));
    w = "ab";
    printf("%c",*w);
    por(w);
    printf("%c",*w);

    return 0;
}

then it prints "aa" instead of "ab". I know its probably pretty stupid question, but i don't know what is going and how to solve that issue.

sizeof (char) is per definition always 1. So no need to write it. So incrementing a pointer can be written simply p++ — Patrick Schlüter
– Patrick Schlüter, Commented Nov 15, 2011 at 14:11
exactly, you actually got a memory leak here because you make w point away to the other place than where it is allocated — ComfortablyNumb
– ComfortablyNumb, Commented Nov 15, 2011 at 14:17
There is also a memory leak because free() isn't called anywhere... — Lundin
– Lundin, Commented Nov 15, 2011 at 14:45

ComfortablyNumb · Accepted Answer · 2011-11-15 14:01:14Z

4

In your por function, t will not be changed. You need change it

void por(char **t){
 *t = *t + sizeof(char);
}

and call it with por(&w)

answered Nov 15, 2011 at 14:01

ComfortablyNumb

3,5212 gold badges17 silver badges15 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

kejmil01 Over a year ago

Thank you. Problem solved, but can you explain it ? :) &w - passes adress of pointer, yes ? I cant understand what **t meaning. (sory for my english)

ComfortablyNumb Over a year ago

You need to dig a bit deeper about how function call takes place. Basically, you parameter will be copied in a function, so the original one will not be changed after the call. To change it, you need manipulate its address. Here what you want to change is a pointer, so you need pass the parameter as the address of pointer: the pointer of pointer. Now can you accept my answer so that I earn some reputation. :)

trojanfoe · Accepted Answer · 2011-11-15 14:06:22Z

1

Try this:

static char *por(char *t)
{
    return t + sizeof(char);
}

int main(void)
{
    char *w = "ab";
    printf("%c",*w);
    w = por(w);
    printf("%c",*w);

    return 0;
}

answered Nov 15, 2011 at 14:06

trojanfoe

123k23 gold badges219 silver badges249 bronze badges

Comments

Bazooka · Accepted Answer · 2011-11-15 15:09:47Z

0

you are incrementing the copy which is local to the function.

answered Nov 15, 2011 at 15:09

Bazooka

1,4684 gold badges17 silver badges24 bronze badges

Collectives™ on Stack Overflow

Changing pointer address - function

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related