Rate My Awesome Cryptographic Hash Algorithm


unsigned long hash(unsigned char *str)
{
unsigned int hash = 0;
int c;

while (c = *str++)
{
hash += c;
}

return hash;
}


Spent hours designing it. It is safer than md5!

>>1
You invoke undefined behaviour, when str points one beyond the 0 char. Consider:

foo = malloc(1);
if(foo) {
*foo = 0;
hash(foo); // bam trap representation, UB invoked inside hash()
free(foo);
}


If you were a C masta you'd also know of the subtlety that this isn't UB when you use hash() with arrays. You're allowed to point one past the last element of an array.