Hi! > +static long caps_len; > +static unsigned char short_buf[4]; > +static unsigned char large_buf[OVERSIZE]; These buffers could have been a guarded buffers. Otherwise: Reviewed-by: Cyril Hrubis <chrubis@suse.cz> -- Cyril Hrubis chrubis@suse.cz