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 Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: LEGO <luis.ontanon@xxxxxxxxx>
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