pcbnew, netlist importer hangs if last bracket missed in netlist

[Sergey A. Borshch]

Steps to reproduce:
1) open eeschema, create schematics, File->Export->Export Netlist->Pcbnew->Export Netlist.
2) Open netlist file in text editor, remove trailing bracket, save file.
3) Open pcbnew, File->Import->Netlist, select netlist file.

Expected behavior: Error message about format violation.
Actual behavior: Application hangs with 100% CPU load.

Version info: today's git head:

Application: Pcbnew
Version: (5.99.0-169-gce62941bb), release build
Libraries:
    wxWidgets 3.0.4
Platform: Linux 5.0.0-23-generic x86_64, 64 bit, Little endian, wxGTK
Build Info:
    Build date: Sep 29 2019 13:17:54
    wxWidgets: 3.0.4 (wchar_t,wx containers,compatible with 2.8) GTK+ 2.24
    Boost: 1.65.1
    Compiler: GCC 7.4.0 with C++ ABI 1011

Build settings:
    KICAD_SCRIPTING=OFF
    KICAD_SCRIPTING_MODULES=OFF
    KICAD_SCRIPTING_PYTHON3=OFF
    KICAD_SCRIPTING_WXPYTHON=OFF
    KICAD_SCRIPTING_WXPYTHON_PHOENIX=OFF
    KICAD_SCRIPTING_ACTION_MENU=OFF
    BUILD_GITHUB_PLUGIN=OFF
    KICAD_USE_OCE=OFF
    KICAD_USE_OCC=OFF
    KICAD_SPICE=ON

[Michael Kavanagh (michaelkavanagh)]

Setting as Medium (not High/Critical) as this can be avoided by using the preferred "Update PCB from Schematic" route.

[Jeff Young (jeyjey)]

Bug is also in 5.1 (and 4.0, for that matter).

[KiCad Janitor (kicad-janitor)]

Fixed in revision 890e1c9ea0788108e92f0c317a95cc7338e1d853
https://git.launchpad.net/kicad/patch/?id=890e1c9ea0788108e92f0c317a95cc7338e1d853