Skip to content

Commit d10df7b

Browse files
emembrivesCQ Bot
authored andcommitted
[memory_monitor] Fix a use-after-move
Remove an access to a member of a moved value. BUG=380874024 BUG=392607900 Change-Id: I764b3e65f99b7ca5c106065e743d61cde08d4b19 Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/1194736 Reviewed-by: Guillaume Chelfi <[email protected]> Commit-Queue: Étienne J. Membrives <[email protected]> Fuchsia-Auto-Submit: Étienne J. Membrives <[email protected]>
1 parent 0d082b2 commit d10df7b

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/developer/memory/metrics/capture_strategy.cc

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,8 @@ zx_status_t BaseCaptureStrategy::OnNewProcess(OS& os, Process process, zx::handl
7171
}
7272
TRACE_DURATION_END("memory_metrics", "BaseCaptureStrategy::OnNewProcess::UniqueVMOs");
7373

74-
koid_to_process_[process.koid] = std::move(process);
74+
zx_koid_t process_koid = process.koid;
75+
koid_to_process_[process_koid] = std::move(process);
7576
return ZX_OK;
7677
}
7778

@@ -89,8 +90,10 @@ zx_status_t StarnixCaptureStrategy::OnNewProcess(OS& os, Process process,
8990
if (process_name_ == process.name) {
9091
starnix_jobs_[process.job].kernel_koid = process.koid;
9192
}
92-
process_handles_[process.koid] = std::move(process_handle);
93-
koid_to_process_[process.koid] = std::move(process);
93+
94+
zx_koid_t process_koid = process.koid;
95+
process_handles_[process_koid] = std::move(process_handle);
96+
koid_to_process_[process_koid] = std::move(process);
9497
return ZX_OK;
9598
}
9699

0 commit comments

Comments
 (0)