25be2d40b2f88155cdadad7c8577642bc5a1e848 braney Sat Apr 20 14:12:59 2024 -0700 sessionId's are unsigned, but we were printing them out as signed diff --git src/hg/lib/cartDb.c src/hg/lib/cartDb.c index 756a4e1..6f7ef72 100644 --- src/hg/lib/cartDb.c +++ src/hg/lib/cartDb.c @@ -109,33 +109,33 @@ { useSessionKey = FALSE; } else if (sameString(sessionKey, "autodetect")) { errAbort("brower.sessionKey=autodetect has not implemented yet."); // TODO } } return useSessionKey; } void cartDbSecureId(char *buf, int bufSize, struct cartDb *cartDb) /* Return combined string of session id plus sessionKey in buf if turned on.*/ { if (cartDbUseSessionKey() && !sameString(cartDb->sessionKey,"")) - safef(buf, bufSize, "%d_%s", cartDb->id, cartDb->sessionKey); + safef(buf, bufSize, "%u_%s", cartDb->id, cartDb->sessionKey); else - safef(buf, bufSize, "%d", cartDb->id); + safef(buf, bufSize, "%u", cartDb->id); } unsigned int cartDbParseId(char *id, char **pSessionKey) /* Parse out the numeric id and id_sessionKey string if present. */ { unsigned int result = 0; if (sameString(id,"")) // some users reported blank cookie values. { verbose(1, "cartDbParseId: id with empty string found."); if (pSessionKey) *pSessionKey = NULL; return 0; } char *e = strchr(id, '_'); if (e)