[bug report] nvme-fabrics: reset admin connection for secure concatenation

Dan Carpenter dan.carpenter at linaro.org
Sat Mar 22 00:40:12 PDT 2025


Hello Hannes Reinecke,

This is a semi-automatic email about new static checker warnings.

Commit 104d0e2f6222 ("nvme-fabrics: reset admin connection for secure
concatenation") from Feb 24, 2025, leads to the following Smatch
complaint:

    drivers/nvme/host/tcp.c:2359 nvme_tcp_setup_ctrl()
    warn: variable dereferenced before check 'ctrl->opts' (see line 2355)

drivers/nvme/host/tcp.c
  2354	
  2355		ret = nvme_tcp_configure_admin_queue(ctrl, new);
                                                     ^^^^
ctrl->opts is dereference here

  2356		if (ret)
  2357			return ret;
  2358	
  2359		if (ctrl->opts && ctrl->opts->concat && !ctrl->tls_pskid) {
                    ^^^^^^^^^^
so hopefully this check can be safely removed.

  2360			/* See comments for nvme_tcp_key_revoke_needed() */
  2361			dev_dbg(ctrl->device, "restart admin queue for secure concatenation\n");

regards,
dan carpenter



More information about the Linux-nvme mailing list