Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/external/bsd/dhcpcd/dist/src



Thanks. Yes, I have the core-dump, no we should not remove the line...

Best,

christos

On Jan 27, 2020, at 8:03 AM, Roy Marples <roy%marples.name@localhost> wrote:

On 27/01/2020 09:03, Roy Marples wrote:
On 26/01/2020 22:57, Christos Zoulas wrote:
Module Name:    src
Committed By:    christos
Date:        Sun Jan 26 22:57:52 UTC 2020

Modified Files:
    src/external/bsd/dhcpcd/dist/src: script.c

Log Message:
prevent coredump when state == NULL


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.12 -r1.2 src/external/bsd/dhcpcd/dist/src/script.c
A quick perusal through the code shows that this should not happen. Every time IPV4LL is given as a reason, the state should exist.

There is one code path where there may be no state .... when an IPv4LL address exists and dhcpcd no longer controlls it (ie dhcpcd restarted or a 3rd party added it) then we delete it anyway (for say when we obtain a DHCP address).
We should still run the IPV4LL hook script, but it will be empty.

I've accepted this patch upstream.

Roy

Attachment: signature.asc
Description: Message signed with OpenPGP



Home | Main Index | Thread Index | Old Index