struct cli_state declares an element "int timeout;" but this value is used as a parameter to client_receive_smb and other functions which declare it as unsigned int timeout. Should we change the cli_state structure to have it as unsigned int timeout?