Might also want to reference my original thread
http://forum.pfsense.org/index.php/topic,66234.0.html
since the backed out patch looks remarkably like the one that was proposed there and where I raised my
concerns as to whether comparing the interface address against $vipent['subnet'] will work (I think it
won't).
If possible, the two threads should probably be merged.