It would be better to change the declaration of PACK_UPIFELAPSED_SALT so that it's a char [] instead of a char*, making the sizeof code correct.  (That way, the buffer size is determined at compile-time, rather than by strlen at run-time.)