ath not setting media type? [SOLVED : devd is to blame]
Sam Leffler
sam at errno.com
Wed Oct 19 10:46:51 PDT 2005
Niki Denev wrote:
> I think i found the problem, and it's in devd.
>
> The problem affects only the media/link status notify events because
> there the interface name is supplied in the variable "subsystem",
> which fails to be assigned properly.
>
> I'm not C++ expert but it looks that the duplicated declaration of the
> variable "value" is the problem.
> I wonder why compiler doesn't complain about this... but i may be wrong,
> because i don't know anyting about c++ :)
>
> this patch fixed it for me :
>
> ------------------------------------------------------------------------------------------------
> --- devd.cc.orig Wed Oct 19 19:50:28 2005
> +++ devd.cc Wed Oct 19 19:53:29 2005
> @@ -233,7 +233,7 @@
> // to subsystem if none exists.
> value = c.get_variable("device-name");
> if (value.length() == 0)
> - string value = c.get_variable("subsystem");
> + value = c.get_variable("subsystem");
> if (Dflag)
> fprintf(stderr, "Testing media type of %s against 0x%x\n",
> value.c_str(), _type);
> ------------------------------------------------------------------------------------------------
>
> [http://www.totalterror.net/freebsd/devd.patch]
>
Thank you; this'll get committed shortly.
Sam
More information about the freebsd-current
mailing list