Re: pf state key linking mbuf assert

2017-12-29 Thread Alexandr Nedvedicky
Hello, On Fri, Dec 29, 2017 at 11:04:07PM +0100, Alexander Bluhm wrote: > On Thu, Dec 28, 2017 at 12:46:45AM +0100, Alexandr Nedvedicky wrote: > > I like your change. You have my OK with small change here: > > I have commitet my diff with a small addition. I fixed a memory > leak that I had

Re: pf state key linking mbuf assert

2017-12-29 Thread Alexander Bluhm
On Thu, Dec 28, 2017 at 12:46:45AM +0100, Alexandr Nedvedicky wrote: > I like your change. You have my OK with small change here: I have commitet my diff with a small addition. I fixed a memory leak that I had introduced before. > > -pf_pkt_state_key_ref(struct mbuf *m) > >

Re: pf state key linking

2017-12-27 Thread Alexandr Nedvedicky
Hello, On Tue, Dec 26, 2017 at 10:02:00PM +0100, Alexander Bluhm wrote: > Hi, > > The functions to link the pf state keys always confused me a bit. > I think the new naming and code of the functions is more consistent. > > ok? I like your change. You have my OK with small change here: >

pf state key linking

2017-12-26 Thread Alexander Bluhm
Hi, The functions to link the pf state keys always confused me a bit. I think the new naming and code of the functions is more consistent. ok? bluhm Index: kern/uipc_mbuf.c === RCS file: