Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to HAVE_CRYPTOHASH

2021-01-25 Thread Vladislav Grishenko
Hi,

> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
> because, why not? and applied. Looks like a sensible idea.

Indeed, much better. Thank you

--
Best Regards, Vladislav Grishenko

> -Original Message-
> From: Dnsmasq-discuss  On
> Behalf Of Simon Kelley
> Sent: Monday, January 25, 2021 3:15 AM
> To: dnsmasq-discuss@lists.thekelleys.org.uk
> Subject: Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to
> HAVE_CRYPTOHASH
> 
> On 24/01/2021 14:30, Vladislav Grishenko wrote:
> > Hi,
> >
> >
> >
> > Commit 2024f9729713fd657d65e64c2e4e471baa0a3e5b "Support hash
> function
> > from nettle (only)" has introduced HAVE_NETTLEHASH option (thanks,
Petr!).
> > But, I think, there's no much sense to bind feature name to specific
> > cryptolib because this will require rename or introduce more similar
> > opts for some other cryptolib backend if/when it'll be available (for
> > example in my dnsmasq-openssl fork).
> >
> > If no objections, let's name it "cryptohash" early before 2.84 is out?
> > Sorry, have missed pre-2.83, but it has dns issues so unlikely be
> > widely deployed.
> >
> > Please refer patch attached.
> >
> >
> 
> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
> because, why not? and applied. Looks like a sensible idea.
> 
> 
> 
> Cheers,
> 
> Simon.
> 
> >
> > --
> >
> > Best Regards, Vladislav Grishenko
> >
> >
> >
> >
> > ___
> > Dnsmasq-discuss mailing list
> > Dnsmasq-discuss@lists.thekelleys.org.uk
> > http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> >
> 
> 
> ___
> Dnsmasq-discuss mailing list
> Dnsmasq-discuss@lists.thekelleys.org.uk
> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to HAVE_CRYPTOHASH

2021-01-25 Thread Petr Menšík
Hi Vladislav,

Where is openssl version used anyway? Would it make sense to support
multiple crypto libraries? Why is just nettle support inadequate? Our
crypto team asked me, why is nettle used. It has no independent FIPS
ceritification, so they would like to use different library, like gnutls
or openssl. Is that similar reason to yours?

I would like to remove dependency on hashing function altogether. It is
not required and slows down the requests handling process IMO. It should
be required only when actual cryptography operations are needed. But
lets postpone it after the security updates are solved and without
regressions.

I just did not think long about the name, CRYPTOHASH sound much better.
Thanks!

On 1/25/21 10:53 AM, Vladislav Grishenko wrote:
> Hi,
> 
>> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
>> because, why not? and applied. Looks like a sensible idea.
> 
> Indeed, much better. Thank you
> 
> --
> Best Regards, Vladislav Grishenko
> 
>> -Original Message-
>> From: Dnsmasq-discuss  On
>> Behalf Of Simon Kelley
>> Sent: Monday, January 25, 2021 3:15 AM
>> To: dnsmasq-discuss@lists.thekelleys.org.uk
>> Subject: Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to
>> HAVE_CRYPTOHASH
>>
>> On 24/01/2021 14:30, Vladislav Grishenko wrote:
>>> Hi,
>>>
>>>
>>>
>>> Commit 2024f9729713fd657d65e64c2e4e471baa0a3e5b "Support hash
>> function
>>> from nettle (only)" has introduced HAVE_NETTLEHASH option (thanks,
> Petr!).
>>> But, I think, there's no much sense to bind feature name to specific
>>> cryptolib because this will require rename or introduce more similar
>>> opts for some other cryptolib backend if/when it'll be available (for
>>> example in my dnsmasq-openssl fork).
>>>
>>> If no objections, let's name it "cryptohash" early before 2.84 is out?
>>> Sorry, have missed pre-2.83, but it has dns issues so unlikely be
>>> widely deployed.
>>>
>>> Please refer patch attached.
>>>
>>>
>>
>> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
>> because, why not? and applied. Looks like a sensible idea.
>>
>>
>>
>> Cheers,
>>
>> Simon.
>>
>>>
>>> --
>>>
>>> Best Regards, Vladislav Grishenko
>>>
>>>
>>>
>>>
>>> ___
>>> Dnsmasq-discuss mailing list
>>> Dnsmasq-discuss@lists.thekelleys.org.uk
>>> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
>>>
>>
>>
>> ___
>> Dnsmasq-discuss mailing list
>> Dnsmasq-discuss@lists.thekelleys.org.uk
>> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> 
> 
> ___
> Dnsmasq-discuss mailing list
> Dnsmasq-discuss@lists.thekelleys.org.uk
> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> 

-- 
Petr Menšík
Software Engineer
Red Hat, http://www.redhat.com/
email: pemen...@redhat.com
PGP: DFCF908DB7C87E8E529925BC4931CA5B6C9FC5CB



OpenPGP_signature
Description: OpenPGP digital signature
___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


Re: [Dnsmasq-discuss] RA-acquired address not marked as 'dynamic' with 2.82

2021-01-25 Thread Christian Ehrhardt
On Thu, Sep 17, 2020 at 8:57 AM Christian Ehrhardt
 wrote:
>
> Hi,
> since nothing happened on this topic (e.g. committed to git / further
> discussion) I wondered if you waited for a confirmation.
>
> The first patch by Iain was ignored or not seen, I'd expect both might work.
> Nevertheless in case there was any (hidden) reason I picked the second
> patch suggested at:
> http://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2020q3/014346.html
>
> I built dnsmasq 2.82 with it and ran the various Ubuntu tests against it.
> I wanted to state that this made it fully work again from my POV and
> hope that it can be considered to be applied.
>

@Simon - Re-ping

We now have had 2.83 (released for CVEs) so some activity is happening
again I guess.
So I thought it might be worth to ping on this "old issue with known
fix" once more.

> P.S. I wasn't subscribed before so I can't reply directly, due to that
> it might be broken up in the tree view of the ML archive, sorry for
> that
>
> --
> Christian Ehrhardt
> Staff Engineer, Ubuntu Server
> Canonical Ltd



-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


[Dnsmasq-discuss] Announce: dnsmasq-2.84

2021-01-25 Thread Simon Kelley
Last week's 2.83 release has proved to have a regression. The symptoms
are random log messages reporting "failure to send packet" and the DNS
query associated with this is lost. Retries of the query do not fail, so
the operational effect of this is minimal. To trigger the bug, dnsmasq
has to be under fairly heavy load, and be configured for a mixture or
IPv4 and IPv6 upstream DNS servers or, possibly, be using
--bind-interfaces. To fix this, I've released 2.84, which has the fix
for the regression, and a couple of housekeeping changes.


Get it here:

http://www.thekelleys.org.uk/dnsmasq/dnsmasq-2.84.tar.gz


The released security backport to version 2.80 has also been updated to
fix the regresssion.

http://www.thekelleys.org.uk/dnsmasq/dnspooq-patches/2.80-dnspooq.patch.v2


Cheers,

Simon.


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


Re: [Dnsmasq-discuss] Announce: dnsmasq-2.84

2021-01-25 Thread Lonnie Abelbeck


> Get it here:
> 
> http://www.thekelleys.org.uk/dnsmasq/dnsmasq-2.84.tar.gz

The version string generated is "2.84rc2"

$ cat dnsmasq-2.84/VERSION 
 (HEAD -> master, tag: v2.84rc2, tag: v2.84, origin/master, origin/HEAD)


Lonnie


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


Re: [Dnsmasq-discuss] Announce: dnsmasq-2.84

2021-01-25 Thread Lonnie Abelbeck



> On Jan 25, 2021, at 5:21 PM, Lonnie Abelbeck  
> wrote:
> 
> 
>> Get it here:
>> 
>> http://www.thekelleys.org.uk/dnsmasq/dnsmasq-2.84.tar.gz
> 
> The version string generated is "2.84rc2"
> 
> $ cat dnsmasq-2.84/VERSION 
> (HEAD -> master, tag: v2.84rc2, tag: v2.84, origin/master, origin/HEAD)

This patch solves the problem:

--- dnsmasq-2.84/bld/get-version.orig   2021-01-25 17:35:04.882908877 -0600
+++ dnsmasq-2.84/bld/get-version2021-01-25 17:35:58.658467974 -0600
@@ -28,7 +28,7 @@
  vers=`cat $1/VERSION | sed 's/[(), ]/,/ g' | tr ',' '\n' | grep ^v[0-9]`
 
  if [ $? -eq 0 ]; then
- echo "${vers}" | sort -r | head -n 1 | sed 's/^v//'
+ echo "${vers}" | sort | head -n 1 | sed 's/^v//'
  else
  cat $1/VERSION
  fi



Lonnie


___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss


Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to HAVE_CRYPTOHASH

2021-01-25 Thread Vladislav Grishenko
Hi Petr,

> Where is openssl version used anyway?
In https://asuswrt-merlin.net, embedded software for wireless routers

> Would it make sense to support multiple crypto libraries?
Taking into account existing official support of nettle and required support of 
system openssl (in asuswrt-merlin) - multilib support looks useful.
Current dnsmasq-openssl work is here https://github.com/themiron/dnsmasq

> Why is just nettle support inadequate?
Because of additional ram/flash footprint, libnettle is used only by dnsmasq 
among the rest of firmware packages.

> Our crypto team asked me, why is nettle used.
I believe nettle was picked due openssl licensing incompatibility while initial 
dnssec approach was done with openssl.
Another point is memory usage with openssl, allocations are dynamic so frequent 
allocations/frees are expected unlike mostly-static nettle.
Simon may give more light on this.

> It has no independent FIPS ceritification, so they would like to use 
> different library, like gnutls or openssl. Is that similar reason to yours?
My reasons are above, fortunately certification is not an issue for 3rd party 
project.
As for openssl license, 3.x version is compatible, and 1.x has no license issue 
if used as system library (as we have).
Ggnutls support implementation seems possible for me, almost like openssl, tho 
till this moment I was not really interested (we have no gnutls used in our 
project).

> I just did not think long about the name, CRYPTOHASH sound much better.
> Thanks!
Np

> 
> On 1/25/21 10:53 AM, Vladislav Grishenko wrote:
> > Hi,
> >
> >> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
> >> because, why not? and applied. Looks like a sensible idea.
> >
> > Indeed, much better. Thank you
> >
> > --
> > Best Regards, Vladislav Grishenko
> >
> >> -Original Message-
> >> From: Dnsmasq-discuss
> >>  On Behalf Of Simon
> >> Kelley
> >> Sent: Monday, January 25, 2021 3:15 AM
> >> To: dnsmasq-discuss@lists.thekelleys.org.uk
> >> Subject: Re: [Dnsmasq-discuss] [PATCH] Rename HAVE_NETTLEHASH to
> >> HAVE_CRYPTOHASH
> >>
> >> On 24/01/2021 14:30, Vladislav Grishenko wrote:
> >>> Hi,
> >>>
> >>>
> >>>
> >>> Commit 2024f9729713fd657d65e64c2e4e471baa0a3e5b "Support hash
> >> function
> >>> from nettle (only)" has introduced HAVE_NETTLEHASH option (thanks,
> > Petr!).
> >>> But, I think, there's no much sense to bind feature name to specific
> >>> cryptolib because this will require rename or introduce more similar
> >>> opts for some other cryptolib backend if/when it'll be available
> >>> (for example in my dnsmasq-openssl fork).
> >>>
> >>> If no objections, let's name it "cryptohash" early before 2.84 is out?
> >>> Sorry, have missed pre-2.83, but it has dns issues so unlikely be
> >>> widely deployed.
> >>>
> >>> Please refer patch attached.
> >>>
> >>>
> >>
> >> Patch modified to keep backwards compatibility with HAVE_NETTLEHASH
> >> because, why not? and applied. Looks like a sensible idea.
> >>
> >>
> >>
> >> Cheers,
> >>
> >> Simon.
> >>
> >>>
> >>> --
> >>>
> >>> Best Regards, Vladislav Grishenko
> >>>
> >>>
> >>>
> >>>
> >>> ___
> >>> Dnsmasq-discuss mailing list
> >>> Dnsmasq-discuss@lists.thekelleys.org.uk
> >>> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> >>>
> >>
> >>
> >> ___
> >> Dnsmasq-discuss mailing list
> >> Dnsmasq-discuss@lists.thekelleys.org.uk
> >> http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> >
> >
> > ___
> > Dnsmasq-discuss mailing list
> > Dnsmasq-discuss@lists.thekelleys.org.uk
> > http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss
> >
> 
> --
> Petr Menšík
> Software Engineer
> Red Hat, http://www.redhat.com/
> email: pemen...@redhat.com
> PGP: DFCF908DB7C87E8E529925BC4931CA5B6C9FC5CB



___
Dnsmasq-discuss mailing list
Dnsmasq-discuss@lists.thekelleys.org.uk
http://lists.thekelleys.org.uk/mailman/listinfo/dnsmasq-discuss