Bug Summary

File:src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/lldb/source/Plugins/Process/minidump/MinidumpTypes.cpp
Warning:line 73, column 10
2nd function call argument is an uninitialized value

Annotated Source Code

Press '?' to see keyboard shortcuts

clang -cc1 -cc1 -triple amd64-unknown-openbsd7.0 -analyze -disable-free -disable-llvm-verifier -discard-value-names -main-file-name MinidumpTypes.cpp -analyzer-store=region -analyzer-opt-analyze-nested-blocks -analyzer-checker=core -analyzer-checker=apiModeling -analyzer-checker=unix -analyzer-checker=deadcode -analyzer-checker=cplusplus -analyzer-checker=security.insecureAPI.UncheckedReturn -analyzer-checker=security.insecureAPI.getpw -analyzer-checker=security.insecureAPI.gets -analyzer-checker=security.insecureAPI.mktemp -analyzer-checker=security.insecureAPI.mkstemp -analyzer-checker=security.insecureAPI.vfork -analyzer-checker=nullability.NullPassedToNonnull -analyzer-checker=nullability.NullReturnedFromNonnull -analyzer-output plist -w -setup-static-analyzer -mrelocation-model static -mframe-pointer=all -relaxed-aliasing -fno-rounding-math -mconstructor-aliases -munwind-tables -target-cpu x86-64 -tune-cpu generic -debugger-tuning=gdb -fcoverage-compilation-dir=/usr/src/gnu/usr.bin/clang/liblldbPluginProcess/obj -resource-dir /usr/local/lib/clang/13.0.0 -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/llvm/include -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../include -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/obj -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/obj/../include -D NDEBUG -D __STDC_LIMIT_MACROS -D __STDC_CONSTANT_MACROS -D __STDC_FORMAT_MACROS -D LLVM_PREFIX="/usr" -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/lldb/include -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/lldb/source -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/clang/include -I /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/obj/../include/lldb/Plugins -internal-isystem /usr/include/c++/v1 -internal-isystem /usr/local/lib/clang/13.0.0/include -internal-externc-isystem /usr/include -O2 -Wno-unused-parameter -Wwrite-strings -Wno-missing-field-initializers -Wno-long-long -Wno-comment -std=c++14 -fdeprecated-macro -fdebug-compilation-dir=/usr/src/gnu/usr.bin/clang/liblldbPluginProcess/obj -ferror-limit 19 -fvisibility-inlines-hidden -fwrapv -stack-protector 2 -fno-rtti -fgnuc-version=4.2.1 -vectorize-loops -vectorize-slp -fno-builtin-malloc -fno-builtin-calloc -fno-builtin-realloc -fno-builtin-valloc -fno-builtin-free -fno-builtin-strdup -fno-builtin-strndup -analyzer-output=html -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -o /home/ben/Projects/vmm/scan-build/2022-01-12-194120-40624-1 -x c++ /usr/src/gnu/usr.bin/clang/liblldbPluginProcess/../../../llvm/lldb/source/Plugins/Process/minidump/MinidumpTypes.cpp
1//===-- MinidumpTypes.cpp -------------------------------------------------===//
2//
3// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4// See https://llvm.org/LICENSE.txt for license information.
5// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6//
7//===----------------------------------------------------------------------===//
8
9#include "MinidumpTypes.h"
10
11// C includes
12// C++ includes
13
14using namespace lldb_private;
15using namespace minidump;
16
17// MinidumpMiscInfo
18const MinidumpMiscInfo *MinidumpMiscInfo::Parse(llvm::ArrayRef<uint8_t> &data) {
19 const MinidumpMiscInfo *misc_info;
20 Status error = consumeObject(data, misc_info);
21 if (error.Fail())
22 return nullptr;
23
24 return misc_info;
25}
26
27llvm::Optional<lldb::pid_t> MinidumpMiscInfo::GetPid() const {
28 uint32_t pid_flag = static_cast<uint32_t>(MinidumpMiscInfoFlags::ProcessID);
29 if (flags1 & pid_flag)
30 return llvm::Optional<lldb::pid_t>(process_id);
31
32 return llvm::None;
33}
34
35// Linux Proc Status
36// it's stored as an ascii string in the file
37llvm::Optional<LinuxProcStatus>
38LinuxProcStatus::Parse(llvm::ArrayRef<uint8_t> &data) {
39 LinuxProcStatus result;
40 result.proc_status =
41 llvm::StringRef(reinterpret_cast<const char *>(data.data()), data.size());
42 data = data.drop_front(data.size());
43
44 llvm::SmallVector<llvm::StringRef, 0> lines;
45 result.proc_status.split(lines, '\n', 42);
46 // /proc/$pid/status has 41 lines, but why not use 42?
47 for (auto line : lines) {
48 if (line.consume_front("Pid:")) {
49 line = line.trim();
50 if (!line.getAsInteger(10, result.pid))
51 return result;
52 }
53 }
54
55 return llvm::None;
56}
57
58lldb::pid_t LinuxProcStatus::GetPid() const { return pid; }
59
60std::pair<llvm::ArrayRef<MinidumpMemoryDescriptor64>, uint64_t>
61MinidumpMemoryDescriptor64::ParseMemory64List(llvm::ArrayRef<uint8_t> &data) {
62 const llvm::support::ulittle64_t *mem_ranges_count;
63 Status error = consumeObject(data, mem_ranges_count);
64 if (error.Fail() ||
1
Assuming the condition is false
→
3
←
Taking false branch
→
65 *mem_ranges_count * sizeof(MinidumpMemoryDescriptor64) > data.size())
2
←
Assuming the condition is false
→
66 return {};
67
68 const llvm::support::ulittle64_t *base_rva;
69 error = consumeObject(data, base_rva);
70 if (error.Fail())
4
←
Assuming the condition is false
→
5
←
Taking false branch
→
71 return {};
72
73 return std::make_pair(
6
←
2nd function call argument is an uninitialized value
74 llvm::makeArrayRef(
75 reinterpret_cast<const MinidumpMemoryDescriptor64 *>(data.data()),
76 *mem_ranges_count),
77 *base_rva);
78}