> On Mon, Mar 18, 2013 at 2:10 PM, Alon Levy <al...@redhat.com> wrote:
> > +    if (len > 2 + historical_length + opt_bytes) {
> > +        DPRINTF(card, D_WARN,
> > +            "atr too long: len %d, but hist/opt %d/%d, T1 0x%X\n",
> > +            len, historical_length, opt_bytes, data[1]);
> > +        /* let it through */
> 
> Why "let it through" if it's too long?

I know windows doesn't like too short, but no idea about too long. Haven't seen 
this in practice yet (well, we control the other end too, but I saw no reason 
to fail it at this point).

> 
> 
> --
> Marc-André Lureau
> 
> 

Reply via email to