strtol reusing param

Question

This code seems to work as expected, populates an array of numbers using a single pointer

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

int main(void)
{
    int arr[4], count = 0, i;
    char *p, s[32] = "  \t  10,  15  \n  ,20,   25  , ";

    p = s;
    do {
        arr[count++] = (int)strtol(p, &p, 10);
        while (isspace(*p) || *p == ',') p++;
    } while (*p);
    for (i = 0; i < count; i++) {
        printf("%d\n", arr[i]);
    }
    return 0;
}

My question is:

It is valid to use p as param1 (source) and &p as param 2 (address of the first invalid character) in strtol?

Klas Lindbäck · Accepted Answer · 2012-12-22 16:29:13Z

6

Yes, it is safe. The first argument is passed by value, so strtol has a local copy that isn't affected by changes written to the second parameter.

edited Dec 22, 2012 at 16:29

user283145

answered Dec 19, 2012 at 12:16

Klas Lindbäck

33.5k5 gold badges63 silver badges85 bronze badges

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

2 Comments

David Ranieri Over a year ago

Thank you Klas, but what about the restrict keywords? <wikipedia> for the lifetime of the pointer, only it or a value directly derived from it (such as pointer + 1) will be used to access the object to which it points, that seems to be a problem

Klas Lindbäck Over a year ago

It would be a problem if strtol would access **endptr, but it doesn't (for the reasons I mentioned in my comment to Kirilenko's answer).

Naresh · Accepted Answer · 2013-09-12 19:59:39Z

1

Yes it is safe.

Please refer to http://en.cppreference.com/w/cpp/string/byte/strtol for complete usage reference. Line 11 of the example illustrates a call using the same variable for the 1st and 2nd parameters.

edited Sep 12, 2013 at 19:59

user283145

answered Dec 19, 2012 at 12:10

Naresh

6781 gold badge7 silver badges22 bronze badges

1 Comment

Jakub Mendyk Over a year ago

As of May 5th 2020, example in en.cppreference.com/w/cpp/string/byte/strtol does not use the same variable for the 1st and 2nd parameters: const long i = std::strtol(p, &p_end, 10);.

Nemanja Boric · Accepted Answer · 2012-12-19 12:06:08Z

Yes, this is valid, as you are keeping the pointer to the beginning of the string (pointer s). Consider that you have this situation:

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

int main(void)
{
    int arr[4], count = 0, i;
    char *p, *s;
    s = (char*)malloc(sizeof(char) * 15);
    strcpy(s, "  \t  10,  15  \n  ,20,   25  , ");

    p = s;
    do {
        arr[count++] = (int)strtol(p, &p, 10);
        while (isspace(*p) || *p == ',') p++;
    } while (*p);
    for (i = 0; i < count; i++) {
        printf("%d\n", arr[i]);
    }
    free(s);
    return 0;
}

strtol will move p pointer to somewhere in string. If you call free(p) you will have memory leak (if it doesn't fail). But, since you are keeping s pointer, you will always be able to free the occupied memory.

Collectives™ on Stack Overflow

strtol reusing param

3 Answers 3

2 Comments

1 Comment

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

Comments

Linked

Related