On Mon, Apr 28, 2014 at 10:56:25AM +0200, Pablo Neira Ayuso wrote:
> > @@ -526,28 +569,93 @@ static int nfacct_cmd_help(int argc, char *argv[])
> >
> > static int nfacct_cmd_restore(int argc, char *argv[])
> > {
> > - uint64_t pkts, bytes;
> > - char name[512];
> > - char buffer[512];
> > - int ret;
> > + uint64_t pkts, bytes, quota;
> > + char name[512], mode[512], buffer[512];
> > + int ret, flags;
> >
> > while (fgets(buffer, sizeof(buffer), stdin)) {
> > char *semicolon = strchr(buffer, ';');
> > +
> > if (semicolon == NULL) {
> > nfacct_perror("invalid line");
> > return -1;
> > }
> > +
> > + /* a single ';' terminates the input */
> > + if (strncmp(buffer, ";", 1) == 0)
> > + break;
>
> This chunk seems new, why is it needed there?
I have applied this patch, we can clarify this or remove it in a
follow up patch.
--
To unsubscribe from this list: send the line "unsubscribe netfilter" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
