<html>
    <head>
      <base href="https://bugzilla.netfilter.org/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Usage message does not mention [file] argument"
   href="https://bugzilla.netfilter.org/show_bug.cgi?id=1341">1341</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Usage message does not mention [file] argument
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>iptables
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>CVS (please indicate timestamp)
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>trivial
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>P5
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>iptables-restore
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>netfilter-buglog@lists.netfilter.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>matt@meinwald.name
          </td>
        </tr></table>
      <p>
        <div>
        <pre>The optional [file] argument is documented in the manpage, but is not present
in the usage message as of the latest commit I see
(093cec72e7f7793d55fa1547cadb6655104feb5e).

It looks like this feature has been available for quite some time, but was only
added to the manpage in 2015. I propose the following trivial update to bring
the usage message in sync as well:

diff --git a/iptables/xtables-restore.c b/iptables/xtables-restore.c
index 86f6a3a..13b9320 100644
--- a/iptables/xtables-restore.c
+++ b/iptables/xtables-restore.c
@@ -43,7 +43,7 @@ static const struct option options[] = {

 static void print_usage(const char *name, const char *version)
 {
-    fprintf(stderr, "Usage: %s [-c] [-v] [-V] [-t] [-h] [-n] [-T table] [-M
command] [-4] [-6]\n"
+    fprintf(stderr, "Usage: %s [-c] [-v] [-V] [-t] [-h] [-n] [-T table] [-M
command] [-4] [-6] [file]\n"
             "       [ --counters ]\n"
             "       [ --verbose ]\n"
             "       [ --version]\n"</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are watching all bug changes.</li>
      </ul>
    </body>
</html>