Patch for ULOG save

Patch for ULOG save

[Jimmy Hedman]

[-- Attachment #1: Type: text/plain, Size: 1027 bytes --]

Hi,
i found that ULOG target doesn't get saved right when doing a
iptables-save so i created a small patch for it. The problem was that it
wasn't putting "'s around ulog-prefix arguments.
Patch attached is against 1.2.7a but the cvs tree seem to have the same
problem.

I don't subscribe to the developer mailinglist so if you answer, please
cc: me.

// Jimmy


Patch:
--- extensions/libipt_ULOG.c.old	2002-09-13 17:12:11 +0200
+++ extensions/libipt_ULOG.c	2002-09-13 17:12:34 +0200
@@ -157,7 +157,7 @@
 	    = (const struct ipt_ulog_info *) target->data;
 
 	if (strcmp(loginfo->prefix, "") != 0)
-		printf("--ulog-prefix %s ", loginfo->prefix);
+		printf("--ulog-prefix \"%s\" ", loginfo->prefix);
 
 	if (loginfo->nl_group != ULOG_DEFAULT_NLGROUP) {
 		printf("--ulog-nlgroup ");


-- 
Jimmy Hedman                       South Pole AB
Phone:  +46 8 51420420             Gelbjutarvägen 5
Fax:    +46 8 51420429             SE - 17148 Solna 
e-mail: jimmy.hedman@southpole.se  www.southpole.se

[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 445 bytes --]

--- extensions/libipt_ULOG.c.old	2002-09-13 17:12:11.000000000 +0200
+++ extensions/libipt_ULOG.c	2002-09-13 17:12:34.000000000 +0200
@@ -157,7 +157,7 @@
 	    = (const struct ipt_ulog_info *) target->data;
 
 	if (strcmp(loginfo->prefix, "") != 0)
-		printf("--ulog-prefix %s ", loginfo->prefix);
+		printf("--ulog-prefix \"%s\" ", loginfo->prefix);
 
 	if (loginfo->nl_group != ULOG_DEFAULT_NLGROUP) {
 		printf("--ulog-nlgroup ");

Re: Patch for ULOG save

[Harald Welte]

[-- Attachment #1: Type: text/plain, Size: 714 bytes --]

On Fri, Sep 13, 2002 at 05:21:49PM +0200, Jimmy Hedman wrote:
> Hi,
> i found that ULOG target doesn't get saved right when doing a
> iptables-save so i created a small patch for it. The problem was that it
> wasn't putting "'s around ulog-prefix arguments.
> Patch attached is against 1.2.7a but the cvs tree seem to have the same
> problem.

thanks, patch applied.

-- 
Live long and prosper
- Harald Welte / laforge@gnumonks.org               http://www.gnumonks.org/
============================================================================
GCS/E/IT d- s-: a-- C+++ UL++++$ P+++ L++++$ E--- W- N++ o? K- w--- O- M+ 
V-- PS++ PE-- Y++ PGP++ t+ 5-- !X !R tv-- b+++ !DI !D G+ e* h--- r++ y+(*)

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]