Re: [PATCH net 3/3] openvswitch: Serialize nested ct actions if provided

2015-10-16 Thread Joe Stringer
On 15 October 2015 at 11:13, Pravin Shelar wrote: > On Wed, Oct 14, 2015 at 11:10 AM, Joe Stringer wrote: >> If userspace provides a ct action with no nested mark or label, then the >> storage for these fields is zeroed. Later when actions are requested, >> such zeroed fields are serialized even

Re: [PATCH net 3/3] openvswitch: Serialize nested ct actions if provided

2015-10-15 Thread Pravin Shelar
On Wed, Oct 14, 2015 at 11:10 AM, Joe Stringer wrote: > If userspace provides a ct action with no nested mark or label, then the > storage for these fields is zeroed. Later when actions are requested, > such zeroed fields are serialized even though userspace didn't > originally specify them. Fix t

[PATCH net 3/3] openvswitch: Serialize nested ct actions if provided

2015-10-14 Thread Joe Stringer
If userspace provides a ct action with no nested mark or label, then the storage for these fields is zeroed. Later when actions are requested, such zeroed fields are serialized even though userspace didn't originally specify them. Fix the behaviour by ensuring that no action is serialized in this c