All of lore.kernel.org
 help / color / mirror / Atom feed
* [LARTC] [PATCH] Problem with wrong using of qdisc/filter in tc
@ 2003-07-18 21:31 Jaroslaw J. Pyszny
  0 siblings, 0 replies; only message in thread
From: Jaroslaw J. Pyszny @ 2003-07-18 21:31 UTC (permalink / raw)
  To: lartc

[-- Attachment #1: Type: text/plain, Size: 1018 bytes --]

Hi,

  I found bug in tc (wrong using of qdisc/filter will cause seg fault):
   tc qdisc add u32 help        // u32 is filter
  
  In function get_qdisc_kind (get_filter_kind) we search for struct
  qdisc_util (filter_util) with name *_util (in example for u32 is 
  u32_util - it's ok, but it isn't qdisc)

  My suggestion is to change name to:
   *_utilQ (for qdisc)
   *_utilF (for filter)

  then if we search for qdisc we won't get filter (and vice versa).
  See patch.
  
  I've tested it on version 2.4.7 (ss010824 and ss020116-try), but the bug
  should also affect older versions.
   
  I wrote to A. Kuznetsov, but I didn't get response. Maybe my email
  lost or something else ;)

  Best regards
  Jarek
-- 
8  \\|||//  Jaroslaw J. Pyszny (arghil AT tlen DOT pl)      \\|||//
b   (o|o)         "Vitae,non scholae,discimus"               (o|o)
i  --\_/--  Linux user: #96704 (http://counter.li.org)      --\_/-- 
t =-> Masz 8-bit. komputer i chcesz się go pozbyc - daj znac. <-=

[-- Attachment #2: iproute2-2.4.7-fix_names.patch.bz2 --]
[-- Type: application/octet-stream, Size: 1458 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2003-07-18 21:31 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-07-18 21:31 [LARTC] [PATCH] Problem with wrong using of qdisc/filter in tc Jaroslaw J. Pyszny

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.