>> + sprintf( win->irdaServiceName, "LSAP-SEL%d", uin->sir_lsap_sel ); > I have no idea if this can even be an issue in practice, but > "uin->sir_lsap_sel" is unsigned. It can't be an issue in practice since LSAP selectors are limited to 0-9, but I can change that. Thanks for the review. --Juan