Bug Summary

File:build/source/lldb/source/Plugins/Process/minidump/MinidumpTypes.cpp
Warning:line 74, column 10
2nd function call argument is an uninitialized value

Annotated Source Code

Press '?' to see keyboard shortcuts

clang -cc1 -cc1 -triple x86_64-pc-linux-gnu -analyze -disable-free -clear-ast-before-backend -disable-llvm-verifier -discard-value-names -main-file-name MinidumpTypes.cpp -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 -analyzer-config-compatibility-mode=true -mrelocation-model pic -pic-level 2 -mframe-pointer=none -fmath-errno -ffp-contract=on -fno-rounding-math -mconstructor-aliases -funwind-tables=2 -target-cpu x86-64 -tune-cpu generic -debugger-tuning=gdb -ffunction-sections -fdata-sections -fcoverage-compilation-dir=/build/source/build-llvm/tools/clang/stage2-bins -resource-dir /usr/lib/llvm-17/lib/clang/17 -isystem /usr/include/libxml2 -D HAVE_ROUND -D _DEBUG -D _GLIBCXX_ASSERTIONS -D _GNU_SOURCE -D _LIBCPP_ENABLE_ASSERTIONS -D __STDC_CONSTANT_MACROS -D __STDC_FORMAT_MACROS -D __STDC_LIMIT_MACROS -I tools/lldb/source/Plugins/Process/minidump -I /build/source/lldb/source/Plugins/Process/minidump -I /build/source/lldb/include -I tools/lldb/include -I include -I /build/source/llvm/include -I /usr/include/python3.9 -I /build/source/clang/include -I tools/lldb/../clang/include -I /build/source/lldb/source -I tools/lldb/source -D _FORTIFY_SOURCE=2 -D NDEBUG -U NDEBUG -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/10/../../../../include/c++/10 -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/10/../../../../include/x86_64-linux-gnu/c++/10 -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/10/../../../../include/c++/10/backward -internal-isystem /usr/lib/llvm-17/lib/clang/17/include -internal-isystem /usr/local/include -internal-isystem /usr/lib/gcc/x86_64-linux-gnu/10/../../../../x86_64-linux-gnu/include -internal-externc-isystem /usr/include/x86_64-linux-gnu -internal-externc-isystem /include -internal-externc-isystem /usr/include -fmacro-prefix-map=/build/source/build-llvm/tools/clang/stage2-bins=build-llvm/tools/clang/stage2-bins -fmacro-prefix-map=/build/source/= -fcoverage-prefix-map=/build/source/build-llvm/tools/clang/stage2-bins=build-llvm/tools/clang/stage2-bins -fcoverage-prefix-map=/build/source/= -source-date-epoch 1683717183 -O2 -Wno-unused-command-line-argument -Wno-unused-parameter -Wwrite-strings -Wno-missing-field-initializers -Wno-long-long -Wno-maybe-uninitialized -Wno-class-memaccess -Wno-redundant-move -Wno-pessimizing-move -Wno-noexcept-type -Wno-comment -Wno-misleading-indentation -Wno-deprecated-declarations -Wno-unknown-pragmas -Wno-strict-aliasing -Wno-stringop-truncation -std=c++17 -fdeprecated-macro -fdebug-compilation-dir=/build/source/build-llvm/tools/clang/stage2-bins -fdebug-prefix-map=/build/source/build-llvm/tools/clang/stage2-bins=build-llvm/tools/clang/stage2-bins -fdebug-prefix-map=/build/source/= -ferror-limit 19 -fvisibility-inlines-hidden -stack-protector 2 -fgnuc-version=4.2.1 -fcolor-diagnostics -vectorize-loops -vectorize-slp -analyzer-output=html -analyzer-config stable-report-filename=true -faddrsig -D__GCC_HAVE_DWARF2_CFI_ASM=1 -o /tmp/scan-build-2023-05-10-133810-16478-1 -x c++ /build/source/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#include <optional>
11
12// C includes
13// C++ includes
14
15using namespace lldb_private;
16using namespace minidump;
17
18// MinidumpMiscInfo
19const MinidumpMiscInfo *MinidumpMiscInfo::Parse(llvm::ArrayRef<uint8_t> &data) {
20 const MinidumpMiscInfo *misc_info;
21 Status error = consumeObject(data, misc_info);
22 if (error.Fail())
23 return nullptr;
24
25 return misc_info;
26}
27
28std::optional<lldb::pid_t> MinidumpMiscInfo::GetPid() const {
29 uint32_t pid_flag = static_cast<uint32_t>(MinidumpMiscInfoFlags::ProcessID);
30 if (flags1 & pid_flag)
31 return std::optional<lldb::pid_t>(process_id);
32
33 return std::nullopt;
34}
35
36// Linux Proc Status
37// it's stored as an ascii string in the file
38std::optional<LinuxProcStatus>
39LinuxProcStatus::Parse(llvm::ArrayRef<uint8_t> &data) {
40 LinuxProcStatus result;
41 result.proc_status =
42 llvm::StringRef(reinterpret_cast<const char *>(data.data()), data.size());
43 data = data.drop_front(data.size());
44
45 llvm::SmallVector<llvm::StringRef, 0> lines;
46 result.proc_status.split(lines, '\n', 42);
47 // /proc/$pid/status has 41 lines, but why not use 42?
48 for (auto line : lines) {
49 if (line.consume_front("Pid:")) {
50 line = line.trim();
51 if (!line.getAsInteger(10, result.pid))
52 return result;
53 }
54 }
55
56 return std::nullopt;
57}
58
59lldb::pid_t LinuxProcStatus::GetPid() const { return pid; }
60
61std::pair<llvm::ArrayRef<MinidumpMemoryDescriptor64>, uint64_t>
62MinidumpMemoryDescriptor64::ParseMemory64List(llvm::ArrayRef<uint8_t> &data) {
63 const llvm::support::ulittle64_t *mem_ranges_count;
64 Status error = consumeObject(data, mem_ranges_count);
65 if (error.Fail() ||
1
Assuming the condition is false
→
3
←
Taking false branch
→
66 *mem_ranges_count * sizeof(MinidumpMemoryDescriptor64) > data.size())
2
←
Assuming the condition is false
→
67 return {};
68
69 const llvm::support::ulittle64_t *base_rva;
70 error = consumeObject(data, base_rva);
71 if (error.Fail())
4
←
Assuming the condition is false
→
5
←
Taking false branch
→
72 return {};
73
74 return std::make_pair(
6
←
2nd function call argument is an uninitialized value
75 llvm::ArrayRef(
76 reinterpret_cast<const MinidumpMemoryDescriptor64 *>(data.data()),
77 *mem_ranges_count),
78 *base_rva);
79}