lkml.org 
[lkml]   [2001]   [Jul]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ppp_generic.c: last_channel_index
Date
From
> get an Oops.  PPP daemon would try to allocate strange PPP interface numbers
> (ppp12, for example) and other sanity checks would fail (kind != INTERFACE
> || CHANNEL)
>
> The last_channel_index wasn't being cleaned up when the channel was closed,
> leading to these problems the next time a PPP channel was opened. This
> patch fixes the problem for me.

Im not convinced this is the actual problem I must admit

> spin_lock_bh(&all_channels_lock);
> + last_channel_index--;
> list_del(&pch->file.list);
> spin_unlock_bh(&all_channels_lock);

Suppose you open channel 0, then 1, then 2, then close 0. Your last_channel_index
is now wrong.

Certainly we should be reusing channel ids but I dont think its as simple as
your patch. The obvious question therefore is why does it work

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:57    [W:0.054 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site