libldap/cyrus.c: rc >> 16 * 0xff (ITS#2729)

[h.b.furuseth@usit.uio.no]

Full_Name: Hallvard B Furuseth
Version: HEAD
OS: 
URL: 
Submission from: (NULL) (129.240.186.42)
Submitted by: hallvard


I imagine 'rc >> 16 * 0xff' in libldap/cyrus.c should
be '(rc >> 16) & 0xff' or (rc >> 16) * 0xff'.  The current code
means rc >> (16 * 0xff), which doesn't look too good.

Also, I suspect rc >> 24 should be (unsigned)rc >> 24, printed with %u.