Re: [PATCH v2] thunderbolt: Do not enumerate more ports from DROM than the controller has

From: Christian Kellner
Date: Tue Jul 25 2017 - 12:05:43 EST


Hi everyone,

+ /*
+ * Some DROMs list more ports than the controller actually has
+ * so we skip those but allow the parser to continue.
+ */
+ if (header->index > sw->config.max_port_number) {
+ dev_info_once(&sw->dev, "ignoring unnecessary extra entries in DROM\n");
+ return 0;
+ }
+
port = &sw->ports[header->index];
port->disabled = header->port_disabled;
if (port->disabled)

Fixes the bug, everything works as expected (tested on boot,
plugging in, key based authorization), so:

Tested-by: Christian Kellner <ckellner@xxxxxxxxxx>