Re: CVS commit: src/sys

[Matthias Drochner <M.Drochner%fz-juelich.de@localhost>]

On Thu, 2 Aug 2012 02:15:08 +0100
Mindaugas Rasiukevicius <rmind%netbsd.org@localhost> wrote:
> "Mindaugas Rasiukevicius" <rmind%netbsd.org@localhost> wrote:
> > Module Name:        src
> > Committed By:       rmind
> > Date:               Wed Aug  1 23:24:29 UTC 2012
> >
> > <...>
> >
> > Log Message:
> [...]
> > Also, make few BPF fixes and
> > simplifications while here.
> [...]

You've added a memset() to clear the scratch memory in
bpf_filter(). We had this before... the point of
bpf_validate() is to make sure that the filter program
doesn't access uninitialized values. The runtime cost
of clearing the array can thus be avoided.
Or do you have reason to believe that the validator
doesn't work as expected?

best regards
Matthias


------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------
Forschungszentrum Juelich GmbH
52425 Juelich
Sitz der Gesellschaft: Juelich
Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
Vorsitzender des Aufsichtsrats: MinDir Dr. Karl Eugen Huthmacher
Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
Karsten Beneke (stellv. Vorsitzender), Prof. Dr.-Ing. Harald Bolt,
Prof. Dr. Sebastian M. Schmidt
------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------

Kennen Sie schon unsere app? http://www.fz-juelich.de/app