Re: panic after r244584

From: Alexander Motin <mav_at_FreeBSD.org>
Date: Fri, 18 Jan 2013 17:10:58 +0200
On 18.01.2013 15:49, Vitalij Satanivskij wrote:
> May be just do sanitizing for elmpriv->descr?
> 
> something like change whitespace to "_" or just delete it?

Yes, that is not difficult. The only question is how to stay consistent,
compatible, user-readable.

> Vitalij Satanivskij wrote:
> VS> Jaakko Heinonen wrote:
> VS> JH> On 2013-01-18, Alexander Motin wrote:
> VS> JH> > > AM> > V>  panic: make_dev_alias_v: bad si_name (error=22 si_name=enc_at_n5003048000bab37d/tpe0/slot_at_1/elmdesc_at_Slot 01/pass7)
> VS> JH> 
> VS> JH> > > AM> The panic is triggered by the check added by the recent r244584 change.
> VS> JH> > > AM> The space in device name came from the enclosure device, and I guess it
> VS> JH> > > AM> may be quite often situation. Using human readable name supposed to help
> VS> JH> > > AM> system administrators, but with spaces banned that may be a problem.
> VS> JH> > > 
> VS> JH> > > That's was not created by human, it was generated (I think so) by system. 
> VS> JH> > 
> VS> JH> > These strings are flashed into enclosure firmware by manufacturer.
> VS> JH> 
> VS> JH> You can't rely on that any string can be safely used as a device name
> VS> JH> even if spaces were allowed. Consider for example duplicate names and
> VS> JH> "../".
> VS> JH> 
> VS> JH> Where these names are generated? The original report didn't contain a
> VS> JH> backtrace.
> VS> 
> VS> Yes. No backtrace, because of switching off all debuging in kernel. 
> VS> 
> VS> For now I can't use that's server for testing, but there are another servers waiting for upgrade.
> VS> 
> VS> I will try to reproduce problem with  kernel debuger enabled.
> VS> 
> VS> 
> VS> _______________________________________________
> VS> freebsd-current_at_freebsd.org mailing list
> VS> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> VS> To unsubscribe, send any mail to "freebsd-current-unsubscribe_at_freebsd.org"
> 


-- 
Alexander Motin
Received on Fri Jan 18 2013 - 14:11:03 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:34 UTC