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