On 03/02/2016 14:39, "Ben Pfaff" <[email protected]> wrote:

>On Wed, Feb 03, 2016 at 02:38:00PM -0800, Ben Pfaff wrote:
>> On Tue, Feb 02, 2016 at 05:56:34PM -0800, Daniele Di Proietto wrote:
>> > This effectively stops vswitchd from creating bridges with '/' in the
>> > name. OVS used to print a warning but the bridge was created anyway.
>> > 
>> > This restriction is implemented because the bridge name is part of a
>> > filesystem path.
>> > 
>> > This check is no substitute for Mandatory Access Control, but it
>> > certainly helps to catch the error early.
>> > 
>> > Signed-off-by: Daniele Di Proietto <[email protected]>
>> 
>> Please add a test.  You can take this one:
>
>Oh, with a test:
>Acked-by: Ben Pfaff <[email protected]>

I didn't think it was possible to write a unit test for this. I added it
and push this up to branch-2.3.

Thanks!

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to