[ldns-users] ldns-verify-zone: General memory error at 5 if -k is missing

[Willem Toorop]

Of course, it is an argument parsing bug.  ldns-verify-zone evaluates
only one single zonefile, so it should report its usage when more then
one zonefile is given as an argument.  I have committed that change to
the developers branch.  Thanks!

-- Willem

op 30-04-14 14:52, Emil Natan schreef:
> Tested with the development version and it really reports "no SOA in the
> zone", but in this case it's misleading because the zonefile actually
> has SOA record and as I mentioned, the problem was a missing flag (-k)
> before the second key-file parameter.
> 
> Emil
> 
> 
> On Wed, Apr 30, 2014 at 3:29 PM, Willem Toorop <willem at nlnetlabs.nl
> <mailto:willem at nlnetlabs.nl>> wrote:
> 
>     Well the version on the develop branch in git reports a descent error
>     message, so I can assure you that ldns 1.6.18 (or ldns 1.7.0) will not
>     have this "bug" anymore..
> 
>     $ ldns-verify-zone -S -k Kinline.test.+007+53625.key
>     Kinline.test.+007+64652.key signed.db.bind
>     ; Error: no SOA in the zone
> 
> 
>     Cheers,
> 
>     -- Willem
> 
>     op 30-04-14 14:10, Emil Natan schreef:
>     > Just paid attention, the command line is missing "-k" before the
>     second
>     > key file is provided. So
>     > $ ldns-verify-zone -S -k Kinline.test.+007+53625.key -k
>     > Kinline.test.+007+64652.key signed.db.bind
>     > works.
>     >
>     >
>     > On Wed, Apr 30, 2014 at 2:46 PM, Emil Natan <shlyoko at gmail.com
>     <mailto:shlyoko at gmail.com>
>     > <mailto:shlyoko at gmail.com <mailto:shlyoko at gmail.com>>> wrote:
>     >
>     >     The same using 1.6.17.
>     >
>     >     Emil
>     >
>     >
>     >     On Wed, Apr 23, 2014 at 6:58 PM, Petr Spacek
>     <pspacek at redhat.com <mailto:pspacek at redhat.com>
>     >     <mailto:pspacek at redhat.com <mailto:pspacek at redhat.com>>> wrote:
>     >
>     >         Hello list,
>     >
>     >         $ ldns-verify-zone -S -k Kinline.test.+007+53625.key
>     >         Kinline.test.+007+64652.key signed.db.bind
>     >
>     >         fails mysteriously with error message:
>     >
>     >         General memory error at 5
>     >
>     >         This is triggered by missing "-k" before second key name, but
>     >         the error message doesn't seem correct...
>     >
>     >         $ ldns-version
>     >         1.6.16
>     >
>     >         $ rpm -q ldns
>     >         ldns-1.6.16-6.fc20.x86_64
>     >
>     >         I'm attaching key and zone files in reproducer.tar.bz2.
>     >
>     >
>     >         Have a nice day!
>     >
>     >         --
>     >         Petr Spacek  @  Red Hat
>     >
>     >         _______________________________________________
>     >         ldns-users mailing list
>     >         ldns-users at open.nlnetlabs.nl
>     <mailto:ldns-users at open.nlnetlabs.nl>
>     <mailto:ldns-users at open.nlnetlabs.nl
>     <mailto:ldns-users at open.nlnetlabs.nl>>
>     >         http://open.nlnetlabs.nl/mailman/listinfo/ldns-users
>     >
>     >
>     >
>     >
>     >
>     > _______________________________________________
>     > ldns-users mailing list
>     > ldns-users at open.nlnetlabs.nl <mailto:ldns-users at open.nlnetlabs.nl>
>     > http://open.nlnetlabs.nl/mailman/listinfo/ldns-users
>     >
> 
>     _______________________________________________
>     ldns-users mailing list
>     ldns-users at open.nlnetlabs.nl <mailto:ldns-users at open.nlnetlabs.nl>
>     http://open.nlnetlabs.nl/mailman/listinfo/ldns-users
> 
>