On Sun, Jul 21, 2019 at 02:14:05AM +0200, Florian Westphal wrote:
> This should never happen (we should pass valid locations to the error
> reporting functions), but in case we screw up we will segfault during
> error reporting.
>
> cat crash
> table inet filter {
> }
> table inet filter {
> chain test {
> counter
> }
> }
> "nft -f crash" Now reports:
> internal:0:0-0: Error: No such file or directory
>
> ... which is both bogus and useless, but better than crashing.
I'd suggest we add BUG() here, so we catch missing location
information via indesc == NULL. So we can fix the lack of it,
otherwise users will rely on internal, which is very limited.
Thanks.