JoaoBR wrote:
On Sunday 01 April 2007 15:22, Mike Tancsa wrote:
At 11:55 AM 4/1/2007, JoaoBR wrote:
by all respect to Julians work but with ipfw broken and sunday fucked up
...
kind of scaring when seeing "I have no time to check, I do it on tuesday"
or "I need to do the userland ipfw too to add some new features, but, not
today.."
please do it all or don't do it, ipfw is an mature and essential
part where we
do not espect such sudden surprises in releng6 to happen
I seriously doubt he intentionally meant to break it.... Accidents
man sure not, no one said that
I thought I had these MFC's all worked out and one seems to have some how been
mixed up in the testing.
Unfortunately the timing really sucked as I was just heading out
when I found that out and I didn't have time to figure out exactly what
went wrong in the MFC. I just backed out what appeared to be the problem commit.
turns out there was another part of it.
This is the missing change that was buried on another commit.
Apparently, the following is also needed, to revert correctly.
--- src/sys/netinet/ip_fw2.c.orig Mon Apr 2 11:48:03 2007
+++ src/sys/netinet/ip_fw2.c Mon Nov 20 18:19:10 2006
@@ -3861,7 +3836,7 @@
case O_PIPE:
case O_QUEUE:
- if (cmdlen != F_INSN_SIZE(ipfw_insn))
+ if (cmdlen != F_INSN_SIZE(ipfw_insn_pipe))
goto bad_size;
goto check_action;
I'll commit this and then forward change the MFC again as soon as I find out
where
the ball was dropped in the MFC testing.
happen. Roll your sources back to Friday and you will be OK until
yaya
but essential and especially mature code should be tested before comitting
changes I guess, I believe that ipfw wasn't tested before beeing hacked and
comitted this time, and overall btw, there was an alert and reply to the
commit msg on cvs which then was politly ignored "until tuesday" ... luck
that it wasn't the bootstrap or something
its sorted out. Remember, its a best effort, not perfect effort project.
sure, but when became perfect the honor is welcome as it comes for free when
it went wrong ;)
---Mike
João
A mensagem foi scaneada pelo sistema de e-mail e pode ser considerada segura.
Service fornecido pelo Datacenter Matik https://datacenter.matik.com.br
_______________________________________________
freebsd-stable@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to "[EMAIL PROTECTED]"