Ethereal-dev: Re: [Ethereal-dev] Re: rev 15998: /trunk/epan/dissectors/: packet-radius.c

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date: Sun, 25 Sep 2005 12:09:59 +0200
which is preety much what we do...

As there's nothing left to do after dissecting the avps.

It's just doing it without  noise.

On 9/25/05, ronnie sahlberg <ronniesahlberg@xxxxxxxxx> wrote:
> I dont think we should just return here.
>
> If the length is invalid there is no point in continuing dissecting
> the pacekt at all  and we should abort dissection completely, not just
> end dissection of the attributes and then continue.
>
>
> On 9/24/05, lego@xxxxxxxxxxxx <lego@xxxxxxxxxxxx> wrote:
> > User: lego
> > Date: 2005/09/24 09:48 PM
> >
> > Log:
> >  Don't ASSERT if a VSA is too short, add an item reporting the error and
> > leave.
> >
> > Directory: /trunk/epan/dissectors/
> >   Changes    Path               Action
> >   +7 -1      packet-radius.c    Modified
> >
> >
> >
> > http://anonsvn.ethereal.com/viewcvs/viewcvs.py?rev=15998&view=rev
> >
> > _______________________________________________
> > Ethereal-cvs mailing list
> > Ethereal-cvs@xxxxxxxxxxxx
> > http://www.ethereal.com/mailman/listinfo/ethereal-cvs
> >
>
> _______________________________________________
> Ethereal-dev mailing list
> Ethereal-dev@xxxxxxxxxxxx
> http://www.ethereal.com/mailman/listinfo/ethereal-dev
>


--
This information is top security. When you have read it, destroy yourself.
-- Marshall McLuhan