Ethereal-dev: [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: ronnie sahlberg <ronniesahlberg@xxxxxxxxx>
Date: Sun, 25 Sep 2005 01:48:54 -0400
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
>