Thread: segment fault related to strlen.S

I was wondering why the marked line in this function always gave "segmentation fault" error at the end of run. No warning at compiling, though.

Code:

void get_hashes(unsigned int hash[],  const char *in)
{
    int pos = strlen(in);            //causing Segmentation fault at end of run!
    hash[0] = RSHash(in, pos);
    hash[1] = DJBHash(in, pos);
    hash[2] = FNVHash(in, pos);
    hash[3] = JSHash(in, pos);
    hash[4] = PJWHash(in, pos);
    hash[5] = SDBMHash(in, pos);
    hash[6] = DEKHash(in, pos);
    hash[7] = murmur(in, (uint64_t) pos, (uint64_t) pos);
}

And when I tried GDB to debug, the error message is:

Code:

Program received signal SIGSEGV, Segmentation fault.
strlen () at ../sysdeps/x86_64/strlen.S:106
106    ../sysdeps/x86_64/strlen.S: No such file or directory.

Can anybody help me with it?
Thanks a lot!

Thanks Matt!
What I am passing to the function for "in" is a string.
A test code was attached.

Note that while it appears you're declaring an array with 22 strings, you have a mistake such that there are only 21. Therefore, when you pass the last element of the array to the "get_hashes()" function, you are accessing the array out of bounds.

The mistake is, when initializing the array, you left out a comma at the end of one of the rows. In C, adjacent string literals are concatenated during compilation. So by not having that comma, what appears to be two separate strings is actually one string.

Note that this behavior of automatic string concatenation can be used to your advantage, so you do not have to use excessively long lines.

For instance, as an example with a single string, this:

Code:

const char *str = "CCAACAGAATACCGATGATTCCCGTCTTCACCTGCCTGTTTTGCCTTTTCCTGGCATCGTTTCTTGCGGAAATGGCGCAGACCGACATGGGATATTTTATTCCTTGAGTTCTGGTC"

... is equivalent to this:

Code:

const char *str = "CCAACAGAATACCGATGATTCCCGTCTTC"
                  "ACCTGCCTGTTTTGCCTTTTCCTGGCATC"
                  "GTTTCTTGCGGAAATGGCGCAGACCGACA"
                  "TGGGATATTTTATTCCTTGAGTTCTGGTC";

Note there are no commas between the strings, to ensure they are concatenated by the compiler.