tree 31fd42b3019e5a155717ed3921933e893c756ee4
parent 567e3ab77c0770da7def4352808e2744aa13369e
author Ed Tanous <edtanous@google.com> 1632509744 -0700
committer Ed Tanous <ed@tanous.net> 1633019475 +0000

Fix use after move in account service

A clang analyzer was able to find this issue.

dbus::utility::checkDbusPathExists(
        dbusObjectPath,
        [dbusObjectPath(std::move(dbusObjectPath)), username,
         password(std::move(password)), roleId(std::move(roleId)), enabled,
         locked, asyncResp{std::move(asyncResp)}](int rc) {

"Object 'dbusObjectPath' of type 'std::basic_string' is left in a valid
but unspecified state after move"

This is obviously a mistake, but apparently the undefined behavior works
for us.  This commit makes it so we no longer rely on UB.

Tested: Trivial change, code compiles.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1b82778cb74f4e5abc53edec35a6d276161de1af