xref: /openbsd-src/gnu/llvm/clang/lib/StaticAnalyzer/Checkers/StreamChecker.cpp (revision 12c855180aad702bbcca06e0398d774beeafb155)
1e5dd7070Spatrick //===-- StreamChecker.cpp -----------------------------------------*- C++ -*--//
2e5dd7070Spatrick //
3e5dd7070Spatrick // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
4e5dd7070Spatrick // See https://llvm.org/LICENSE.txt for license information.
5e5dd7070Spatrick // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
6e5dd7070Spatrick //
7e5dd7070Spatrick //===----------------------------------------------------------------------===//
8e5dd7070Spatrick //
9e5dd7070Spatrick // This file defines checkers that model and check stream handling functions.
10e5dd7070Spatrick //
11e5dd7070Spatrick //===----------------------------------------------------------------------===//
12e5dd7070Spatrick 
13e5dd7070Spatrick #include "clang/StaticAnalyzer/Checkers/BuiltinCheckerRegistration.h"
14e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
15e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/Checker.h"
16e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/CheckerManager.h"
17*12c85518Srobert #include "clang/StaticAnalyzer/Core/PathSensitive/CallDescription.h"
18e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
19e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
20*12c85518Srobert #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerHelpers.h"
21e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h"
22e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/PathSensitive/ProgramStateTrait.h"
23e5dd7070Spatrick #include "clang/StaticAnalyzer/Core/PathSensitive/SymbolManager.h"
24e5dd7070Spatrick #include <functional>
25*12c85518Srobert #include <optional>
26e5dd7070Spatrick 
27e5dd7070Spatrick using namespace clang;
28e5dd7070Spatrick using namespace ento;
29e5dd7070Spatrick using namespace std::placeholders;
30e5dd7070Spatrick 
31a9ac8606Spatrick //===----------------------------------------------------------------------===//
32a9ac8606Spatrick // Definition of state data structures.
33a9ac8606Spatrick //===----------------------------------------------------------------------===//
34a9ac8606Spatrick 
35e5dd7070Spatrick namespace {
36e5dd7070Spatrick 
37ec727ea7Spatrick struct FnDescription;
38e5dd7070Spatrick 
39ec727ea7Spatrick /// State of the stream error flags.
40ec727ea7Spatrick /// Sometimes it is not known to the checker what error flags are set.
41ec727ea7Spatrick /// This is indicated by setting more than one flag to true.
42ec727ea7Spatrick /// This is an optimization to avoid state splits.
43ec727ea7Spatrick /// A stream can either be in FEOF or FERROR but not both at the same time.
44ec727ea7Spatrick /// Multiple flags are set to handle the corresponding states together.
45ec727ea7Spatrick struct StreamErrorState {
46ec727ea7Spatrick   /// The stream can be in state where none of the error flags set.
47ec727ea7Spatrick   bool NoError = true;
48ec727ea7Spatrick   /// The stream can be in state where the EOF indicator is set.
49ec727ea7Spatrick   bool FEof = false;
50ec727ea7Spatrick   /// The stream can be in state where the error indicator is set.
51ec727ea7Spatrick   bool FError = false;
52e5dd7070Spatrick 
isNoError__anon6d5dc6460111::StreamErrorState53ec727ea7Spatrick   bool isNoError() const { return NoError && !FEof && !FError; }
isFEof__anon6d5dc6460111::StreamErrorState54ec727ea7Spatrick   bool isFEof() const { return !NoError && FEof && !FError; }
isFError__anon6d5dc6460111::StreamErrorState55ec727ea7Spatrick   bool isFError() const { return !NoError && !FEof && FError; }
56e5dd7070Spatrick 
operator ==__anon6d5dc6460111::StreamErrorState57ec727ea7Spatrick   bool operator==(const StreamErrorState &ES) const {
58ec727ea7Spatrick     return NoError == ES.NoError && FEof == ES.FEof && FError == ES.FError;
59ec727ea7Spatrick   }
60e5dd7070Spatrick 
operator !=__anon6d5dc6460111::StreamErrorState61ec727ea7Spatrick   bool operator!=(const StreamErrorState &ES) const { return !(*this == ES); }
62ec727ea7Spatrick 
operator |__anon6d5dc6460111::StreamErrorState63ec727ea7Spatrick   StreamErrorState operator|(const StreamErrorState &E) const {
64ec727ea7Spatrick     return {NoError || E.NoError, FEof || E.FEof, FError || E.FError};
65ec727ea7Spatrick   }
66ec727ea7Spatrick 
operator &__anon6d5dc6460111::StreamErrorState67ec727ea7Spatrick   StreamErrorState operator&(const StreamErrorState &E) const {
68ec727ea7Spatrick     return {NoError && E.NoError, FEof && E.FEof, FError && E.FError};
69ec727ea7Spatrick   }
70ec727ea7Spatrick 
operator ~__anon6d5dc6460111::StreamErrorState71ec727ea7Spatrick   StreamErrorState operator~() const { return {!NoError, !FEof, !FError}; }
72ec727ea7Spatrick 
73ec727ea7Spatrick   /// Returns if the StreamErrorState is a valid object.
operator bool__anon6d5dc6460111::StreamErrorState74ec727ea7Spatrick   operator bool() const { return NoError || FEof || FError; }
75e5dd7070Spatrick 
Profile__anon6d5dc6460111::StreamErrorState76e5dd7070Spatrick   void Profile(llvm::FoldingSetNodeID &ID) const {
77ec727ea7Spatrick     ID.AddBoolean(NoError);
78ec727ea7Spatrick     ID.AddBoolean(FEof);
79ec727ea7Spatrick     ID.AddBoolean(FError);
80e5dd7070Spatrick   }
81e5dd7070Spatrick };
82e5dd7070Spatrick 
83ec727ea7Spatrick const StreamErrorState ErrorNone{true, false, false};
84ec727ea7Spatrick const StreamErrorState ErrorFEof{false, true, false};
85ec727ea7Spatrick const StreamErrorState ErrorFError{false, false, true};
86e5dd7070Spatrick 
87ec727ea7Spatrick /// Full state information about a stream pointer.
88ec727ea7Spatrick struct StreamState {
89ec727ea7Spatrick   /// The last file operation called in the stream.
90*12c85518Srobert   /// Can be nullptr.
91ec727ea7Spatrick   const FnDescription *LastOperation;
92e5dd7070Spatrick 
93ec727ea7Spatrick   /// State of a stream symbol.
94ec727ea7Spatrick   enum KindTy {
95ec727ea7Spatrick     Opened, /// Stream is opened.
96ec727ea7Spatrick     Closed, /// Closed stream (an invalid stream pointer after it was closed).
97ec727ea7Spatrick     OpenFailed /// The last open operation has failed.
98ec727ea7Spatrick   } State;
99e5dd7070Spatrick 
100ec727ea7Spatrick   /// State of the error flags.
101ec727ea7Spatrick   /// Ignored in non-opened stream state but must be NoError.
102ec727ea7Spatrick   StreamErrorState const ErrorState;
103ec727ea7Spatrick 
104ec727ea7Spatrick   /// Indicate if the file has an "indeterminate file position indicator".
105ec727ea7Spatrick   /// This can be set at a failing read or write or seek operation.
106ec727ea7Spatrick   /// If it is set no more read or write is allowed.
107ec727ea7Spatrick   /// This value is not dependent on the stream error flags:
108ec727ea7Spatrick   /// The error flag may be cleared with `clearerr` but the file position
109ec727ea7Spatrick   /// remains still indeterminate.
110ec727ea7Spatrick   /// This value applies to all error states in ErrorState except FEOF.
111ec727ea7Spatrick   /// An EOF+indeterminate state is the same as EOF state.
112ec727ea7Spatrick   bool const FilePositionIndeterminate = false;
113ec727ea7Spatrick 
StreamState__anon6d5dc6460111::StreamState114ec727ea7Spatrick   StreamState(const FnDescription *L, KindTy S, const StreamErrorState &ES,
115ec727ea7Spatrick               bool IsFilePositionIndeterminate)
116ec727ea7Spatrick       : LastOperation(L), State(S), ErrorState(ES),
117ec727ea7Spatrick         FilePositionIndeterminate(IsFilePositionIndeterminate) {
118ec727ea7Spatrick     assert((!ES.isFEof() || !IsFilePositionIndeterminate) &&
119ec727ea7Spatrick            "FilePositionIndeterminate should be false in FEof case.");
120ec727ea7Spatrick     assert((State == Opened || ErrorState.isNoError()) &&
121ec727ea7Spatrick            "ErrorState should be None in non-opened stream state.");
122ec727ea7Spatrick   }
123ec727ea7Spatrick 
isOpened__anon6d5dc6460111::StreamState124ec727ea7Spatrick   bool isOpened() const { return State == Opened; }
isClosed__anon6d5dc6460111::StreamState125ec727ea7Spatrick   bool isClosed() const { return State == Closed; }
isOpenFailed__anon6d5dc6460111::StreamState126ec727ea7Spatrick   bool isOpenFailed() const { return State == OpenFailed; }
127ec727ea7Spatrick 
operator ==__anon6d5dc6460111::StreamState128ec727ea7Spatrick   bool operator==(const StreamState &X) const {
129ec727ea7Spatrick     // In not opened state error state should always NoError, so comparison
130ec727ea7Spatrick     // here is no problem.
131ec727ea7Spatrick     return LastOperation == X.LastOperation && State == X.State &&
132ec727ea7Spatrick            ErrorState == X.ErrorState &&
133ec727ea7Spatrick            FilePositionIndeterminate == X.FilePositionIndeterminate;
134ec727ea7Spatrick   }
135ec727ea7Spatrick 
getOpened__anon6d5dc6460111::StreamState136ec727ea7Spatrick   static StreamState getOpened(const FnDescription *L,
137ec727ea7Spatrick                                const StreamErrorState &ES = ErrorNone,
138ec727ea7Spatrick                                bool IsFilePositionIndeterminate = false) {
139ec727ea7Spatrick     return StreamState{L, Opened, ES, IsFilePositionIndeterminate};
140ec727ea7Spatrick   }
getClosed__anon6d5dc6460111::StreamState141ec727ea7Spatrick   static StreamState getClosed(const FnDescription *L) {
142ec727ea7Spatrick     return StreamState{L, Closed, {}, false};
143ec727ea7Spatrick   }
getOpenFailed__anon6d5dc6460111::StreamState144ec727ea7Spatrick   static StreamState getOpenFailed(const FnDescription *L) {
145ec727ea7Spatrick     return StreamState{L, OpenFailed, {}, false};
146ec727ea7Spatrick   }
147ec727ea7Spatrick 
Profile__anon6d5dc6460111::StreamState148ec727ea7Spatrick   void Profile(llvm::FoldingSetNodeID &ID) const {
149ec727ea7Spatrick     ID.AddPointer(LastOperation);
150ec727ea7Spatrick     ID.AddInteger(State);
151*12c85518Srobert     ErrorState.Profile(ID);
152ec727ea7Spatrick     ID.AddBoolean(FilePositionIndeterminate);
153ec727ea7Spatrick   }
154e5dd7070Spatrick };
155e5dd7070Spatrick 
156a9ac8606Spatrick } // namespace
157a9ac8606Spatrick 
158a9ac8606Spatrick //===----------------------------------------------------------------------===//
159a9ac8606Spatrick // StreamChecker class and utility functions.
160a9ac8606Spatrick //===----------------------------------------------------------------------===//
161a9ac8606Spatrick 
162a9ac8606Spatrick namespace {
163a9ac8606Spatrick 
164ec727ea7Spatrick class StreamChecker;
165ec727ea7Spatrick using FnCheck = std::function<void(const StreamChecker *, const FnDescription *,
166ec727ea7Spatrick                                    const CallEvent &, CheckerContext &)>;
167e5dd7070Spatrick 
168ec727ea7Spatrick using ArgNoTy = unsigned int;
169ec727ea7Spatrick static const ArgNoTy ArgNone = std::numeric_limits<ArgNoTy>::max();
170ec727ea7Spatrick 
171ec727ea7Spatrick struct FnDescription {
172ec727ea7Spatrick   FnCheck PreFn;
173ec727ea7Spatrick   FnCheck EvalFn;
174ec727ea7Spatrick   ArgNoTy StreamArgNo;
175ec727ea7Spatrick };
176ec727ea7Spatrick 
177ec727ea7Spatrick /// Get the value of the stream argument out of the passed call event.
178ec727ea7Spatrick /// The call should contain a function that is described by Desc.
getStreamArg(const FnDescription * Desc,const CallEvent & Call)179ec727ea7Spatrick SVal getStreamArg(const FnDescription *Desc, const CallEvent &Call) {
180ec727ea7Spatrick   assert(Desc && Desc->StreamArgNo != ArgNone &&
181ec727ea7Spatrick          "Try to get a non-existing stream argument.");
182ec727ea7Spatrick   return Call.getArgSVal(Desc->StreamArgNo);
183ec727ea7Spatrick }
184ec727ea7Spatrick 
185ec727ea7Spatrick /// Create a conjured symbol return value for a call expression.
makeRetVal(CheckerContext & C,const CallExpr * CE)186ec727ea7Spatrick DefinedSVal makeRetVal(CheckerContext &C, const CallExpr *CE) {
187ec727ea7Spatrick   assert(CE && "Expecting a call expression.");
188ec727ea7Spatrick 
189ec727ea7Spatrick   const LocationContext *LCtx = C.getLocationContext();
190ec727ea7Spatrick   return C.getSValBuilder()
191ec727ea7Spatrick       .conjureSymbolVal(nullptr, CE, LCtx, C.blockCount())
192ec727ea7Spatrick       .castAs<DefinedSVal>();
193ec727ea7Spatrick }
194ec727ea7Spatrick 
bindAndAssumeTrue(ProgramStateRef State,CheckerContext & C,const CallExpr * CE)195ec727ea7Spatrick ProgramStateRef bindAndAssumeTrue(ProgramStateRef State, CheckerContext &C,
196ec727ea7Spatrick                                   const CallExpr *CE) {
197ec727ea7Spatrick   DefinedSVal RetVal = makeRetVal(C, CE);
198ec727ea7Spatrick   State = State->BindExpr(CE, C.getLocationContext(), RetVal);
199ec727ea7Spatrick   State = State->assume(RetVal, true);
200ec727ea7Spatrick   assert(State && "Assumption on new value should not fail.");
201ec727ea7Spatrick   return State;
202ec727ea7Spatrick }
203ec727ea7Spatrick 
bindInt(uint64_t Value,ProgramStateRef State,CheckerContext & C,const CallExpr * CE)204ec727ea7Spatrick ProgramStateRef bindInt(uint64_t Value, ProgramStateRef State,
205ec727ea7Spatrick                         CheckerContext &C, const CallExpr *CE) {
206ec727ea7Spatrick   State = State->BindExpr(CE, C.getLocationContext(),
207*12c85518Srobert                           C.getSValBuilder().makeIntVal(Value, CE->getType()));
208ec727ea7Spatrick   return State;
209ec727ea7Spatrick }
210ec727ea7Spatrick 
211ec727ea7Spatrick class StreamChecker : public Checker<check::PreCall, eval::Call,
212ec727ea7Spatrick                                      check::DeadSymbols, check::PointerEscape> {
213ec727ea7Spatrick   BugType BT_UseAfterClose{this, "Closed stream", "Stream handling error"};
214ec727ea7Spatrick   BugType BT_UseAfterOpenFailed{this, "Invalid stream",
215ec727ea7Spatrick                                 "Stream handling error"};
216ec727ea7Spatrick   BugType BT_IndeterminatePosition{this, "Invalid stream state",
217ec727ea7Spatrick                                    "Stream handling error"};
218ec727ea7Spatrick   BugType BT_IllegalWhence{this, "Illegal whence argument",
219ec727ea7Spatrick                            "Stream handling error"};
220ec727ea7Spatrick   BugType BT_StreamEof{this, "Stream already in EOF", "Stream handling error"};
221a9ac8606Spatrick   BugType BT_ResourceLeak{this, "Resource leak", "Stream handling error",
222a9ac8606Spatrick                           /*SuppressOnSink =*/true};
223ec727ea7Spatrick 
224ec727ea7Spatrick public:
225ec727ea7Spatrick   void checkPreCall(const CallEvent &Call, CheckerContext &C) const;
226ec727ea7Spatrick   bool evalCall(const CallEvent &Call, CheckerContext &C) const;
227ec727ea7Spatrick   void checkDeadSymbols(SymbolReaper &SymReaper, CheckerContext &C) const;
228ec727ea7Spatrick   ProgramStateRef checkPointerEscape(ProgramStateRef State,
229ec727ea7Spatrick                                      const InvalidatedSymbols &Escaped,
230ec727ea7Spatrick                                      const CallEvent *Call,
231ec727ea7Spatrick                                      PointerEscapeKind Kind) const;
232ec727ea7Spatrick 
233ec727ea7Spatrick   /// If true, evaluate special testing stream functions.
234ec727ea7Spatrick   bool TestMode = false;
235ec727ea7Spatrick 
getBT_StreamEof() const236a9ac8606Spatrick   const BugType *getBT_StreamEof() const { return &BT_StreamEof; }
237a9ac8606Spatrick 
238ec727ea7Spatrick private:
239ec727ea7Spatrick   CallDescriptionMap<FnDescription> FnDescriptions = {
240*12c85518Srobert       {{{"fopen"}}, {nullptr, &StreamChecker::evalFopen, ArgNone}},
241*12c85518Srobert       {{{"freopen"}, 3},
242ec727ea7Spatrick        {&StreamChecker::preFreopen, &StreamChecker::evalFreopen, 2}},
243*12c85518Srobert       {{{"tmpfile"}}, {nullptr, &StreamChecker::evalFopen, ArgNone}},
244*12c85518Srobert       {{{"fclose"}, 1},
245ec727ea7Spatrick        {&StreamChecker::preDefault, &StreamChecker::evalFclose, 0}},
246*12c85518Srobert       {{{"fread"}, 4},
247ec727ea7Spatrick        {&StreamChecker::preFread,
248ec727ea7Spatrick         std::bind(&StreamChecker::evalFreadFwrite, _1, _2, _3, _4, true), 3}},
249*12c85518Srobert       {{{"fwrite"}, 4},
250ec727ea7Spatrick        {&StreamChecker::preFwrite,
251ec727ea7Spatrick         std::bind(&StreamChecker::evalFreadFwrite, _1, _2, _3, _4, false), 3}},
252*12c85518Srobert       {{{"fseek"}, 3},
253*12c85518Srobert        {&StreamChecker::preFseek, &StreamChecker::evalFseek, 0}},
254*12c85518Srobert       {{{"ftell"}, 1},
255*12c85518Srobert        {&StreamChecker::preDefault, &StreamChecker::evalFtell, 0}},
256*12c85518Srobert       {{{"rewind"}, 1},
257*12c85518Srobert        {&StreamChecker::preDefault, &StreamChecker::evalRewind, 0}},
258*12c85518Srobert       {{{"fgetpos"}, 2},
259*12c85518Srobert        {&StreamChecker::preDefault, &StreamChecker::evalFgetpos, 0}},
260*12c85518Srobert       {{{"fsetpos"}, 2},
261*12c85518Srobert        {&StreamChecker::preDefault, &StreamChecker::evalFsetpos, 0}},
262*12c85518Srobert       {{{"clearerr"}, 1},
263ec727ea7Spatrick        {&StreamChecker::preDefault, &StreamChecker::evalClearerr, 0}},
264*12c85518Srobert       {{{"feof"}, 1},
265ec727ea7Spatrick        {&StreamChecker::preDefault,
266ec727ea7Spatrick         std::bind(&StreamChecker::evalFeofFerror, _1, _2, _3, _4, ErrorFEof),
267ec727ea7Spatrick         0}},
268*12c85518Srobert       {{{"ferror"}, 1},
269ec727ea7Spatrick        {&StreamChecker::preDefault,
270ec727ea7Spatrick         std::bind(&StreamChecker::evalFeofFerror, _1, _2, _3, _4, ErrorFError),
271ec727ea7Spatrick         0}},
272*12c85518Srobert       {{{"fileno"}, 1}, {&StreamChecker::preDefault, nullptr, 0}},
273ec727ea7Spatrick   };
274ec727ea7Spatrick 
275ec727ea7Spatrick   CallDescriptionMap<FnDescription> FnTestDescriptions = {
276*12c85518Srobert       {{{"StreamTesterChecker_make_feof_stream"}, 1},
277ec727ea7Spatrick        {nullptr,
278ec727ea7Spatrick         std::bind(&StreamChecker::evalSetFeofFerror, _1, _2, _3, _4, ErrorFEof),
279ec727ea7Spatrick         0}},
280*12c85518Srobert       {{{"StreamTesterChecker_make_ferror_stream"}, 1},
281ec727ea7Spatrick        {nullptr,
282ec727ea7Spatrick         std::bind(&StreamChecker::evalSetFeofFerror, _1, _2, _3, _4,
283ec727ea7Spatrick                   ErrorFError),
284ec727ea7Spatrick         0}},
285ec727ea7Spatrick   };
286ec727ea7Spatrick 
287*12c85518Srobert   mutable std::optional<int> EofVal;
288*12c85518Srobert 
289ec727ea7Spatrick   void evalFopen(const FnDescription *Desc, const CallEvent &Call,
290ec727ea7Spatrick                  CheckerContext &C) const;
291ec727ea7Spatrick 
292ec727ea7Spatrick   void preFreopen(const FnDescription *Desc, const CallEvent &Call,
293ec727ea7Spatrick                   CheckerContext &C) const;
294ec727ea7Spatrick   void evalFreopen(const FnDescription *Desc, const CallEvent &Call,
295ec727ea7Spatrick                    CheckerContext &C) const;
296ec727ea7Spatrick 
297ec727ea7Spatrick   void evalFclose(const FnDescription *Desc, const CallEvent &Call,
298ec727ea7Spatrick                   CheckerContext &C) const;
299ec727ea7Spatrick 
300ec727ea7Spatrick   void preFread(const FnDescription *Desc, const CallEvent &Call,
301ec727ea7Spatrick                 CheckerContext &C) const;
302ec727ea7Spatrick 
303ec727ea7Spatrick   void preFwrite(const FnDescription *Desc, const CallEvent &Call,
304ec727ea7Spatrick                  CheckerContext &C) const;
305ec727ea7Spatrick 
306ec727ea7Spatrick   void evalFreadFwrite(const FnDescription *Desc, const CallEvent &Call,
307ec727ea7Spatrick                        CheckerContext &C, bool IsFread) const;
308ec727ea7Spatrick 
309ec727ea7Spatrick   void preFseek(const FnDescription *Desc, const CallEvent &Call,
310ec727ea7Spatrick                 CheckerContext &C) const;
311ec727ea7Spatrick   void evalFseek(const FnDescription *Desc, const CallEvent &Call,
312ec727ea7Spatrick                  CheckerContext &C) const;
313ec727ea7Spatrick 
314*12c85518Srobert   void evalFgetpos(const FnDescription *Desc, const CallEvent &Call,
315*12c85518Srobert                    CheckerContext &C) const;
316*12c85518Srobert 
317*12c85518Srobert   void evalFsetpos(const FnDescription *Desc, const CallEvent &Call,
318*12c85518Srobert                    CheckerContext &C) const;
319*12c85518Srobert 
320*12c85518Srobert   void evalFtell(const FnDescription *Desc, const CallEvent &Call,
321*12c85518Srobert                  CheckerContext &C) const;
322*12c85518Srobert 
323*12c85518Srobert   void evalRewind(const FnDescription *Desc, const CallEvent &Call,
324*12c85518Srobert                   CheckerContext &C) const;
325*12c85518Srobert 
326ec727ea7Spatrick   void preDefault(const FnDescription *Desc, const CallEvent &Call,
327ec727ea7Spatrick                   CheckerContext &C) const;
328ec727ea7Spatrick 
329ec727ea7Spatrick   void evalClearerr(const FnDescription *Desc, const CallEvent &Call,
330ec727ea7Spatrick                     CheckerContext &C) const;
331ec727ea7Spatrick 
332ec727ea7Spatrick   void evalFeofFerror(const FnDescription *Desc, const CallEvent &Call,
333ec727ea7Spatrick                       CheckerContext &C,
334ec727ea7Spatrick                       const StreamErrorState &ErrorKind) const;
335ec727ea7Spatrick 
336ec727ea7Spatrick   void evalSetFeofFerror(const FnDescription *Desc, const CallEvent &Call,
337ec727ea7Spatrick                          CheckerContext &C,
338ec727ea7Spatrick                          const StreamErrorState &ErrorKind) const;
339ec727ea7Spatrick 
340ec727ea7Spatrick   /// Check that the stream (in StreamVal) is not NULL.
341*12c85518Srobert   /// If it can only be NULL a sink node is generated and nullptr returned.
342ec727ea7Spatrick   /// Otherwise the return value is a new state where the stream is constrained
343ec727ea7Spatrick   /// to be non-null.
344a9ac8606Spatrick   ProgramStateRef ensureStreamNonNull(SVal StreamVal, const Expr *StreamE,
345a9ac8606Spatrick                                       CheckerContext &C,
346ec727ea7Spatrick                                       ProgramStateRef State) const;
347ec727ea7Spatrick 
348ec727ea7Spatrick   /// Check that the stream is the opened state.
349ec727ea7Spatrick   /// If the stream is known to be not opened an error is generated
350ec727ea7Spatrick   /// and nullptr returned, otherwise the original state is returned.
351ec727ea7Spatrick   ProgramStateRef ensureStreamOpened(SVal StreamVal, CheckerContext &C,
352ec727ea7Spatrick                                      ProgramStateRef State) const;
353ec727ea7Spatrick 
354ec727ea7Spatrick   /// Check that the stream has not an invalid ("indeterminate") file position,
355ec727ea7Spatrick   /// generate warning for it.
356ec727ea7Spatrick   /// (EOF is not an invalid position.)
357ec727ea7Spatrick   /// The returned state can be nullptr if a fatal error was generated.
358ec727ea7Spatrick   /// It can return non-null state if the stream has not an invalid position or
359ec727ea7Spatrick   /// there is execution path with non-invalid position.
360ec727ea7Spatrick   ProgramStateRef
361ec727ea7Spatrick   ensureNoFilePositionIndeterminate(SVal StreamVal, CheckerContext &C,
362ec727ea7Spatrick                                     ProgramStateRef State) const;
363ec727ea7Spatrick 
364ec727ea7Spatrick   /// Check the legality of the 'whence' argument of 'fseek'.
365ec727ea7Spatrick   /// Generate error and return nullptr if it is found to be illegal.
366ec727ea7Spatrick   /// Otherwise returns the state.
367ec727ea7Spatrick   /// (State is not changed here because the "whence" value is already known.)
368ec727ea7Spatrick   ProgramStateRef ensureFseekWhenceCorrect(SVal WhenceVal, CheckerContext &C,
369ec727ea7Spatrick                                            ProgramStateRef State) const;
370ec727ea7Spatrick 
371ec727ea7Spatrick   /// Generate warning about stream in EOF state.
372ec727ea7Spatrick   /// There will be always a state transition into the passed State,
373ec727ea7Spatrick   /// by the new non-fatal error node or (if failed) a normal transition,
374ec727ea7Spatrick   /// to ensure uniform handling.
375a9ac8606Spatrick   void reportFEofWarning(SymbolRef StreamSym, CheckerContext &C,
376a9ac8606Spatrick                          ProgramStateRef State) const;
377a9ac8606Spatrick 
378a9ac8606Spatrick   /// Emit resource leak warnings for the given symbols.
379a9ac8606Spatrick   /// Createn a non-fatal error node for these, and returns it (if any warnings
380a9ac8606Spatrick   /// were generated). Return value is non-null.
381a9ac8606Spatrick   ExplodedNode *reportLeaks(const SmallVector<SymbolRef, 2> &LeakedSyms,
382a9ac8606Spatrick                             CheckerContext &C, ExplodedNode *Pred) const;
383ec727ea7Spatrick 
384ec727ea7Spatrick   /// Find the description data of the function called by a call event.
385ec727ea7Spatrick   /// Returns nullptr if no function is recognized.
lookupFn(const CallEvent & Call) const386ec727ea7Spatrick   const FnDescription *lookupFn(const CallEvent &Call) const {
387ec727ea7Spatrick     // Recognize "global C functions" with only integral or pointer arguments
388ec727ea7Spatrick     // (and matching name) as stream functions.
389ec727ea7Spatrick     if (!Call.isGlobalCFunction())
390ec727ea7Spatrick       return nullptr;
391*12c85518Srobert     for (auto *P : Call.parameters()) {
392ec727ea7Spatrick       QualType T = P->getType();
393ec727ea7Spatrick       if (!T->isIntegralOrEnumerationType() && !T->isPointerType())
394ec727ea7Spatrick         return nullptr;
395ec727ea7Spatrick     }
396ec727ea7Spatrick 
397ec727ea7Spatrick     return FnDescriptions.lookup(Call);
398ec727ea7Spatrick   }
399ec727ea7Spatrick 
400ec727ea7Spatrick   /// Generate a message for BugReporterVisitor if the stored symbol is
401ec727ea7Spatrick   /// marked as interesting by the actual bug report.
402a9ac8606Spatrick   // FIXME: Use lambda instead.
403ec727ea7Spatrick   struct NoteFn {
404a9ac8606Spatrick     const BugType *BT_ResourceLeak;
405ec727ea7Spatrick     SymbolRef StreamSym;
406ec727ea7Spatrick     std::string Message;
407ec727ea7Spatrick 
operator ()__anon6d5dc6460211::StreamChecker::NoteFn408ec727ea7Spatrick     std::string operator()(PathSensitiveBugReport &BR) const {
409a9ac8606Spatrick       if (BR.isInteresting(StreamSym) && &BR.getBugType() == BT_ResourceLeak)
410ec727ea7Spatrick         return Message;
411ec727ea7Spatrick 
412ec727ea7Spatrick       return "";
413ec727ea7Spatrick     }
414ec727ea7Spatrick   };
415ec727ea7Spatrick 
constructNoteTag(CheckerContext & C,SymbolRef StreamSym,const std::string & Message) const416ec727ea7Spatrick   const NoteTag *constructNoteTag(CheckerContext &C, SymbolRef StreamSym,
417ec727ea7Spatrick                                   const std::string &Message) const {
418a9ac8606Spatrick     return C.getNoteTag(NoteFn{&BT_ResourceLeak, StreamSym, Message});
419a9ac8606Spatrick   }
420a9ac8606Spatrick 
constructSetEofNoteTag(CheckerContext & C,SymbolRef StreamSym) const421a9ac8606Spatrick   const NoteTag *constructSetEofNoteTag(CheckerContext &C,
422a9ac8606Spatrick                                         SymbolRef StreamSym) const {
423a9ac8606Spatrick     return C.getNoteTag([this, StreamSym](PathSensitiveBugReport &BR) {
424a9ac8606Spatrick       if (!BR.isInteresting(StreamSym) ||
425a9ac8606Spatrick           &BR.getBugType() != this->getBT_StreamEof())
426a9ac8606Spatrick         return "";
427a9ac8606Spatrick 
428a9ac8606Spatrick       BR.markNotInteresting(StreamSym);
429a9ac8606Spatrick 
430a9ac8606Spatrick       return "Assuming stream reaches end-of-file here";
431a9ac8606Spatrick     });
432ec727ea7Spatrick   }
433ec727ea7Spatrick 
initEof(CheckerContext & C) const434*12c85518Srobert   void initEof(CheckerContext &C) const {
435*12c85518Srobert     if (EofVal)
436*12c85518Srobert       return;
437*12c85518Srobert 
438*12c85518Srobert     if (const std::optional<int> OptInt =
439*12c85518Srobert             tryExpandAsInteger("EOF", C.getPreprocessor()))
440*12c85518Srobert       EofVal = *OptInt;
441*12c85518Srobert     else
442*12c85518Srobert       EofVal = -1;
443*12c85518Srobert   }
444*12c85518Srobert 
445ec727ea7Spatrick   /// Searches for the ExplodedNode where the file descriptor was acquired for
446ec727ea7Spatrick   /// StreamSym.
447ec727ea7Spatrick   static const ExplodedNode *getAcquisitionSite(const ExplodedNode *N,
448ec727ea7Spatrick                                                 SymbolRef StreamSym,
449ec727ea7Spatrick                                                 CheckerContext &C);
450e5dd7070Spatrick };
451e5dd7070Spatrick 
452e5dd7070Spatrick } // end anonymous namespace
453e5dd7070Spatrick 
454a9ac8606Spatrick // This map holds the state of a stream.
455a9ac8606Spatrick // The stream is identified with a SymbolRef that is created when a stream
456a9ac8606Spatrick // opening function is modeled by the checker.
REGISTER_MAP_WITH_PROGRAMSTATE(StreamMap,SymbolRef,StreamState)457e5dd7070Spatrick REGISTER_MAP_WITH_PROGRAMSTATE(StreamMap, SymbolRef, StreamState)
458e5dd7070Spatrick 
459ec727ea7Spatrick inline void assertStreamStateOpened(const StreamState *SS) {
460*12c85518Srobert   assert(SS->isOpened() && "Stream is expected to be opened");
461e5dd7070Spatrick }
462e5dd7070Spatrick 
getAcquisitionSite(const ExplodedNode * N,SymbolRef StreamSym,CheckerContext & C)463ec727ea7Spatrick const ExplodedNode *StreamChecker::getAcquisitionSite(const ExplodedNode *N,
464ec727ea7Spatrick                                                       SymbolRef StreamSym,
465ec727ea7Spatrick                                                       CheckerContext &C) {
466ec727ea7Spatrick   ProgramStateRef State = N->getState();
467ec727ea7Spatrick   // When bug type is resource leak, exploded node N may not have state info
468ec727ea7Spatrick   // for leaked file descriptor, but predecessor should have it.
469ec727ea7Spatrick   if (!State->get<StreamMap>(StreamSym))
470ec727ea7Spatrick     N = N->getFirstPred();
471ec727ea7Spatrick 
472ec727ea7Spatrick   const ExplodedNode *Pred = N;
473ec727ea7Spatrick   while (N) {
474ec727ea7Spatrick     State = N->getState();
475ec727ea7Spatrick     if (!State->get<StreamMap>(StreamSym))
476ec727ea7Spatrick       return Pred;
477ec727ea7Spatrick     Pred = N;
478ec727ea7Spatrick     N = N->getFirstPred();
479ec727ea7Spatrick   }
480ec727ea7Spatrick 
481ec727ea7Spatrick   return nullptr;
482ec727ea7Spatrick }
483ec727ea7Spatrick 
484a9ac8606Spatrick //===----------------------------------------------------------------------===//
485a9ac8606Spatrick // Methods of StreamChecker.
486a9ac8606Spatrick //===----------------------------------------------------------------------===//
487a9ac8606Spatrick 
checkPreCall(const CallEvent & Call,CheckerContext & C) const488ec727ea7Spatrick void StreamChecker::checkPreCall(const CallEvent &Call,
489ec727ea7Spatrick                                  CheckerContext &C) const {
490*12c85518Srobert   initEof(C);
491*12c85518Srobert 
492ec727ea7Spatrick   const FnDescription *Desc = lookupFn(Call);
493ec727ea7Spatrick   if (!Desc || !Desc->PreFn)
494ec727ea7Spatrick     return;
495ec727ea7Spatrick 
496ec727ea7Spatrick   Desc->PreFn(this, Desc, Call, C);
497ec727ea7Spatrick }
498ec727ea7Spatrick 
evalCall(const CallEvent & Call,CheckerContext & C) const499ec727ea7Spatrick bool StreamChecker::evalCall(const CallEvent &Call, CheckerContext &C) const {
500ec727ea7Spatrick   const FnDescription *Desc = lookupFn(Call);
501ec727ea7Spatrick   if (!Desc && TestMode)
502ec727ea7Spatrick     Desc = FnTestDescriptions.lookup(Call);
503ec727ea7Spatrick   if (!Desc || !Desc->EvalFn)
504e5dd7070Spatrick     return false;
505e5dd7070Spatrick 
506ec727ea7Spatrick   Desc->EvalFn(this, Desc, Call, C);
507e5dd7070Spatrick 
508e5dd7070Spatrick   return C.isDifferent();
509e5dd7070Spatrick }
510e5dd7070Spatrick 
evalFopen(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const511ec727ea7Spatrick void StreamChecker::evalFopen(const FnDescription *Desc, const CallEvent &Call,
512ec727ea7Spatrick                               CheckerContext &C) const {
513ec727ea7Spatrick   ProgramStateRef State = C.getState();
514ec727ea7Spatrick   const CallExpr *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
515e5dd7070Spatrick   if (!CE)
516e5dd7070Spatrick     return;
517e5dd7070Spatrick 
518ec727ea7Spatrick   DefinedSVal RetVal = makeRetVal(C, CE);
519ec727ea7Spatrick   SymbolRef RetSym = RetVal.getAsSymbol();
520ec727ea7Spatrick   assert(RetSym && "RetVal must be a symbol here.");
521e5dd7070Spatrick 
522ec727ea7Spatrick   State = State->BindExpr(CE, C.getLocationContext(), RetVal);
523ec727ea7Spatrick 
524e5dd7070Spatrick   // Bifurcate the state into two: one with a valid FILE* pointer, the other
525e5dd7070Spatrick   // with a NULL.
526ec727ea7Spatrick   ProgramStateRef StateNotNull, StateNull;
527ec727ea7Spatrick   std::tie(StateNotNull, StateNull) =
528ec727ea7Spatrick       C.getConstraintManager().assumeDual(State, RetVal);
529e5dd7070Spatrick 
530ec727ea7Spatrick   StateNotNull =
531ec727ea7Spatrick       StateNotNull->set<StreamMap>(RetSym, StreamState::getOpened(Desc));
532ec727ea7Spatrick   StateNull =
533ec727ea7Spatrick       StateNull->set<StreamMap>(RetSym, StreamState::getOpenFailed(Desc));
534e5dd7070Spatrick 
535ec727ea7Spatrick   C.addTransition(StateNotNull,
536ec727ea7Spatrick                   constructNoteTag(C, RetSym, "Stream opened here"));
537ec727ea7Spatrick   C.addTransition(StateNull);
538e5dd7070Spatrick }
539e5dd7070Spatrick 
preFreopen(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const540ec727ea7Spatrick void StreamChecker::preFreopen(const FnDescription *Desc, const CallEvent &Call,
541ec727ea7Spatrick                                CheckerContext &C) const {
542ec727ea7Spatrick   // Do not allow NULL as passed stream pointer but allow a closed stream.
543ec727ea7Spatrick   ProgramStateRef State = C.getState();
544a9ac8606Spatrick   State = ensureStreamNonNull(getStreamArg(Desc, Call),
545a9ac8606Spatrick                               Call.getArgExpr(Desc->StreamArgNo), C, State);
546ec727ea7Spatrick   if (!State)
547ec727ea7Spatrick     return;
548ec727ea7Spatrick 
549ec727ea7Spatrick   C.addTransition(State);
550ec727ea7Spatrick }
551ec727ea7Spatrick 
evalFreopen(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const552ec727ea7Spatrick void StreamChecker::evalFreopen(const FnDescription *Desc,
553ec727ea7Spatrick                                 const CallEvent &Call,
554e5dd7070Spatrick                                 CheckerContext &C) const {
555e5dd7070Spatrick   ProgramStateRef State = C.getState();
556e5dd7070Spatrick 
557e5dd7070Spatrick   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
558e5dd7070Spatrick   if (!CE)
559e5dd7070Spatrick     return;
560e5dd7070Spatrick 
561*12c85518Srobert   std::optional<DefinedSVal> StreamVal =
562ec727ea7Spatrick       getStreamArg(Desc, Call).getAs<DefinedSVal>();
563e5dd7070Spatrick   if (!StreamVal)
564e5dd7070Spatrick     return;
565e5dd7070Spatrick 
566e5dd7070Spatrick   SymbolRef StreamSym = StreamVal->getAsSymbol();
567ec727ea7Spatrick   // Do not care about concrete values for stream ("(FILE *)0x12345"?).
568ec727ea7Spatrick   // FIXME: Can be stdin, stdout, stderr such values?
569e5dd7070Spatrick   if (!StreamSym)
570e5dd7070Spatrick     return;
571e5dd7070Spatrick 
572ec727ea7Spatrick   // Do not handle untracked stream. It is probably escaped.
573ec727ea7Spatrick   if (!State->get<StreamMap>(StreamSym))
574ec727ea7Spatrick     return;
575ec727ea7Spatrick 
576e5dd7070Spatrick   // Generate state for non-failed case.
577e5dd7070Spatrick   // Return value is the passed stream pointer.
578e5dd7070Spatrick   // According to the documentations, the stream is closed first
579e5dd7070Spatrick   // but any close error is ignored. The state changes to (or remains) opened.
580e5dd7070Spatrick   ProgramStateRef StateRetNotNull =
581e5dd7070Spatrick       State->BindExpr(CE, C.getLocationContext(), *StreamVal);
582e5dd7070Spatrick   // Generate state for NULL return value.
583e5dd7070Spatrick   // Stream switches to OpenFailed state.
584*12c85518Srobert   ProgramStateRef StateRetNull =
585*12c85518Srobert       State->BindExpr(CE, C.getLocationContext(),
586*12c85518Srobert                       C.getSValBuilder().makeNullWithType(CE->getType()));
587e5dd7070Spatrick 
588e5dd7070Spatrick   StateRetNotNull =
589ec727ea7Spatrick       StateRetNotNull->set<StreamMap>(StreamSym, StreamState::getOpened(Desc));
590e5dd7070Spatrick   StateRetNull =
591ec727ea7Spatrick       StateRetNull->set<StreamMap>(StreamSym, StreamState::getOpenFailed(Desc));
592e5dd7070Spatrick 
593ec727ea7Spatrick   C.addTransition(StateRetNotNull,
594ec727ea7Spatrick                   constructNoteTag(C, StreamSym, "Stream reopened here"));
595e5dd7070Spatrick   C.addTransition(StateRetNull);
596e5dd7070Spatrick }
597e5dd7070Spatrick 
evalFclose(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const598ec727ea7Spatrick void StreamChecker::evalFclose(const FnDescription *Desc, const CallEvent &Call,
599ec727ea7Spatrick                                CheckerContext &C) const {
600e5dd7070Spatrick   ProgramStateRef State = C.getState();
601ec727ea7Spatrick   SymbolRef Sym = getStreamArg(Desc, Call).getAsSymbol();
602ec727ea7Spatrick   if (!Sym)
603ec727ea7Spatrick     return;
604ec727ea7Spatrick 
605ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(Sym);
606ec727ea7Spatrick   if (!SS)
607ec727ea7Spatrick     return;
608ec727ea7Spatrick 
609*12c85518Srobert   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
610*12c85518Srobert   if (!CE)
611*12c85518Srobert     return;
612*12c85518Srobert 
613ec727ea7Spatrick   assertStreamStateOpened(SS);
614ec727ea7Spatrick 
615ec727ea7Spatrick   // Close the File Descriptor.
616ec727ea7Spatrick   // Regardless if the close fails or not, stream becomes "closed"
617ec727ea7Spatrick   // and can not be used any more.
618ec727ea7Spatrick   State = State->set<StreamMap>(Sym, StreamState::getClosed(Desc));
619ec727ea7Spatrick 
620*12c85518Srobert   // Return 0 on success, EOF on failure.
621*12c85518Srobert   SValBuilder &SVB = C.getSValBuilder();
622*12c85518Srobert   ProgramStateRef StateSuccess = State->BindExpr(
623*12c85518Srobert       CE, C.getLocationContext(), SVB.makeIntVal(0, C.getASTContext().IntTy));
624*12c85518Srobert   ProgramStateRef StateFailure =
625*12c85518Srobert       State->BindExpr(CE, C.getLocationContext(),
626*12c85518Srobert                       SVB.makeIntVal(*EofVal, C.getASTContext().IntTy));
627*12c85518Srobert 
628*12c85518Srobert   C.addTransition(StateSuccess);
629*12c85518Srobert   C.addTransition(StateFailure);
630e5dd7070Spatrick }
631e5dd7070Spatrick 
preFread(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const632ec727ea7Spatrick void StreamChecker::preFread(const FnDescription *Desc, const CallEvent &Call,
633ec727ea7Spatrick                              CheckerContext &C) const {
634e5dd7070Spatrick   ProgramStateRef State = C.getState();
635ec727ea7Spatrick   SVal StreamVal = getStreamArg(Desc, Call);
636a9ac8606Spatrick   State = ensureStreamNonNull(StreamVal, Call.getArgExpr(Desc->StreamArgNo), C,
637a9ac8606Spatrick                               State);
638ec727ea7Spatrick   if (!State)
639ec727ea7Spatrick     return;
640ec727ea7Spatrick   State = ensureStreamOpened(StreamVal, C, State);
641ec727ea7Spatrick   if (!State)
642ec727ea7Spatrick     return;
643ec727ea7Spatrick   State = ensureNoFilePositionIndeterminate(StreamVal, C, State);
644ec727ea7Spatrick   if (!State)
645e5dd7070Spatrick     return;
646e5dd7070Spatrick 
647ec727ea7Spatrick   SymbolRef Sym = StreamVal.getAsSymbol();
648ec727ea7Spatrick   if (Sym && State->get<StreamMap>(Sym)) {
649ec727ea7Spatrick     const StreamState *SS = State->get<StreamMap>(Sym);
650ec727ea7Spatrick     if (SS->ErrorState & ErrorFEof)
651a9ac8606Spatrick       reportFEofWarning(Sym, C, State);
652ec727ea7Spatrick   } else {
653e5dd7070Spatrick     C.addTransition(State);
654ec727ea7Spatrick   }
655e5dd7070Spatrick }
656e5dd7070Spatrick 
preFwrite(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const657ec727ea7Spatrick void StreamChecker::preFwrite(const FnDescription *Desc, const CallEvent &Call,
658ec727ea7Spatrick                               CheckerContext &C) const {
659e5dd7070Spatrick   ProgramStateRef State = C.getState();
660ec727ea7Spatrick   SVal StreamVal = getStreamArg(Desc, Call);
661a9ac8606Spatrick   State = ensureStreamNonNull(StreamVal, Call.getArgExpr(Desc->StreamArgNo), C,
662a9ac8606Spatrick                               State);
663ec727ea7Spatrick   if (!State)
664ec727ea7Spatrick     return;
665ec727ea7Spatrick   State = ensureStreamOpened(StreamVal, C, State);
666ec727ea7Spatrick   if (!State)
667ec727ea7Spatrick     return;
668ec727ea7Spatrick   State = ensureNoFilePositionIndeterminate(StreamVal, C, State);
669ec727ea7Spatrick   if (!State)
670ec727ea7Spatrick     return;
671ec727ea7Spatrick 
672e5dd7070Spatrick   C.addTransition(State);
673e5dd7070Spatrick }
674e5dd7070Spatrick 
evalFreadFwrite(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C,bool IsFread) const675ec727ea7Spatrick void StreamChecker::evalFreadFwrite(const FnDescription *Desc,
676ec727ea7Spatrick                                     const CallEvent &Call, CheckerContext &C,
677ec727ea7Spatrick                                     bool IsFread) const {
678ec727ea7Spatrick   ProgramStateRef State = C.getState();
679ec727ea7Spatrick   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
680ec727ea7Spatrick   if (!StreamSym)
681ec727ea7Spatrick     return;
682ec727ea7Spatrick 
683ec727ea7Spatrick   const CallExpr *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
684ec727ea7Spatrick   if (!CE)
685ec727ea7Spatrick     return;
686ec727ea7Spatrick 
687*12c85518Srobert   std::optional<NonLoc> SizeVal = Call.getArgSVal(1).getAs<NonLoc>();
688ec727ea7Spatrick   if (!SizeVal)
689ec727ea7Spatrick     return;
690*12c85518Srobert   std::optional<NonLoc> NMembVal = Call.getArgSVal(2).getAs<NonLoc>();
691ec727ea7Spatrick   if (!NMembVal)
692ec727ea7Spatrick     return;
693ec727ea7Spatrick 
694a9ac8606Spatrick   const StreamState *OldSS = State->get<StreamMap>(StreamSym);
695a9ac8606Spatrick   if (!OldSS)
696ec727ea7Spatrick     return;
697ec727ea7Spatrick 
698a9ac8606Spatrick   assertStreamStateOpened(OldSS);
699ec727ea7Spatrick 
700ec727ea7Spatrick   // C'99 standard, §7.19.8.1.3, the return value of fread:
701ec727ea7Spatrick   // The fread function returns the number of elements successfully read, which
702ec727ea7Spatrick   // may be less than nmemb if a read error or end-of-file is encountered. If
703ec727ea7Spatrick   // size or nmemb is zero, fread returns zero and the contents of the array and
704ec727ea7Spatrick   // the state of the stream remain unchanged.
705ec727ea7Spatrick 
706ec727ea7Spatrick   if (State->isNull(*SizeVal).isConstrainedTrue() ||
707ec727ea7Spatrick       State->isNull(*NMembVal).isConstrainedTrue()) {
708ec727ea7Spatrick     // This is the "size or nmemb is zero" case.
709ec727ea7Spatrick     // Just return 0, do nothing more (not clear the error flags).
710ec727ea7Spatrick     State = bindInt(0, State, C, CE);
711ec727ea7Spatrick     C.addTransition(State);
712ec727ea7Spatrick     return;
713ec727ea7Spatrick   }
714ec727ea7Spatrick 
715ec727ea7Spatrick   // Generate a transition for the success state.
716ec727ea7Spatrick   // If we know the state to be FEOF at fread, do not add a success state.
717a9ac8606Spatrick   if (!IsFread || (OldSS->ErrorState != ErrorFEof)) {
718ec727ea7Spatrick     ProgramStateRef StateNotFailed =
719ec727ea7Spatrick         State->BindExpr(CE, C.getLocationContext(), *NMembVal);
720*12c85518Srobert     StateNotFailed =
721*12c85518Srobert         StateNotFailed->set<StreamMap>(StreamSym, StreamState::getOpened(Desc));
722ec727ea7Spatrick     C.addTransition(StateNotFailed);
723ec727ea7Spatrick   }
724ec727ea7Spatrick 
725ec727ea7Spatrick   // Add transition for the failed state.
726*12c85518Srobert   NonLoc RetVal = makeRetVal(C, CE).castAs<NonLoc>();
727ec727ea7Spatrick   ProgramStateRef StateFailed =
728*12c85518Srobert       State->BindExpr(CE, C.getLocationContext(), RetVal);
729*12c85518Srobert   auto Cond =
730*12c85518Srobert       C.getSValBuilder()
731*12c85518Srobert           .evalBinOpNN(State, BO_LT, RetVal, *NMembVal, C.getASTContext().IntTy)
732ec727ea7Spatrick           .getAs<DefinedOrUnknownSVal>();
733ec727ea7Spatrick   if (!Cond)
734ec727ea7Spatrick     return;
735ec727ea7Spatrick   StateFailed = StateFailed->assume(*Cond, true);
736ec727ea7Spatrick   if (!StateFailed)
737ec727ea7Spatrick     return;
738ec727ea7Spatrick 
739ec727ea7Spatrick   StreamErrorState NewES;
740ec727ea7Spatrick   if (IsFread)
741a9ac8606Spatrick     NewES =
742a9ac8606Spatrick         (OldSS->ErrorState == ErrorFEof) ? ErrorFEof : ErrorFEof | ErrorFError;
743ec727ea7Spatrick   else
744ec727ea7Spatrick     NewES = ErrorFError;
745ec727ea7Spatrick   // If a (non-EOF) error occurs, the resulting value of the file position
746ec727ea7Spatrick   // indicator for the stream is indeterminate.
747a9ac8606Spatrick   StreamState NewSS = StreamState::getOpened(Desc, NewES, !NewES.isFEof());
748a9ac8606Spatrick   StateFailed = StateFailed->set<StreamMap>(StreamSym, NewSS);
749a9ac8606Spatrick   if (IsFread && OldSS->ErrorState != ErrorFEof)
750a9ac8606Spatrick     C.addTransition(StateFailed, constructSetEofNoteTag(C, StreamSym));
751a9ac8606Spatrick   else
752ec727ea7Spatrick     C.addTransition(StateFailed);
753ec727ea7Spatrick }
754ec727ea7Spatrick 
preFseek(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const755ec727ea7Spatrick void StreamChecker::preFseek(const FnDescription *Desc, const CallEvent &Call,
756ec727ea7Spatrick                              CheckerContext &C) const {
757ec727ea7Spatrick   ProgramStateRef State = C.getState();
758ec727ea7Spatrick   SVal StreamVal = getStreamArg(Desc, Call);
759a9ac8606Spatrick   State = ensureStreamNonNull(StreamVal, Call.getArgExpr(Desc->StreamArgNo), C,
760a9ac8606Spatrick                               State);
761ec727ea7Spatrick   if (!State)
762ec727ea7Spatrick     return;
763ec727ea7Spatrick   State = ensureStreamOpened(StreamVal, C, State);
764ec727ea7Spatrick   if (!State)
765ec727ea7Spatrick     return;
766ec727ea7Spatrick   State = ensureFseekWhenceCorrect(Call.getArgSVal(2), C, State);
767ec727ea7Spatrick   if (!State)
768ec727ea7Spatrick     return;
769ec727ea7Spatrick 
770ec727ea7Spatrick   C.addTransition(State);
771ec727ea7Spatrick }
772ec727ea7Spatrick 
evalFseek(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const773ec727ea7Spatrick void StreamChecker::evalFseek(const FnDescription *Desc, const CallEvent &Call,
774ec727ea7Spatrick                               CheckerContext &C) const {
775ec727ea7Spatrick   ProgramStateRef State = C.getState();
776ec727ea7Spatrick   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
777ec727ea7Spatrick   if (!StreamSym)
778ec727ea7Spatrick     return;
779ec727ea7Spatrick 
780ec727ea7Spatrick   const CallExpr *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
781ec727ea7Spatrick   if (!CE)
782ec727ea7Spatrick     return;
783ec727ea7Spatrick 
784ec727ea7Spatrick   // Ignore the call if the stream is not tracked.
785ec727ea7Spatrick   if (!State->get<StreamMap>(StreamSym))
786ec727ea7Spatrick     return;
787ec727ea7Spatrick 
788ec727ea7Spatrick   DefinedSVal RetVal = makeRetVal(C, CE);
789ec727ea7Spatrick 
790ec727ea7Spatrick   // Make expression result.
791ec727ea7Spatrick   State = State->BindExpr(CE, C.getLocationContext(), RetVal);
792ec727ea7Spatrick 
793ec727ea7Spatrick   // Bifurcate the state into failed and non-failed.
794ec727ea7Spatrick   // Return zero on success, nonzero on error.
795ec727ea7Spatrick   ProgramStateRef StateNotFailed, StateFailed;
796ec727ea7Spatrick   std::tie(StateFailed, StateNotFailed) =
797ec727ea7Spatrick       C.getConstraintManager().assumeDual(State, RetVal);
798ec727ea7Spatrick 
799ec727ea7Spatrick   // Reset the state to opened with no error.
800ec727ea7Spatrick   StateNotFailed =
801ec727ea7Spatrick       StateNotFailed->set<StreamMap>(StreamSym, StreamState::getOpened(Desc));
802ec727ea7Spatrick   // We get error.
803ec727ea7Spatrick   // It is possible that fseek fails but sets none of the error flags.
804ec727ea7Spatrick   // If fseek failed, assume that the file position becomes indeterminate in any
805ec727ea7Spatrick   // case.
806ec727ea7Spatrick   StateFailed = StateFailed->set<StreamMap>(
807ec727ea7Spatrick       StreamSym,
808ec727ea7Spatrick       StreamState::getOpened(Desc, ErrorNone | ErrorFEof | ErrorFError, true));
809ec727ea7Spatrick 
810ec727ea7Spatrick   C.addTransition(StateNotFailed);
811a9ac8606Spatrick   C.addTransition(StateFailed, constructSetEofNoteTag(C, StreamSym));
812ec727ea7Spatrick }
813ec727ea7Spatrick 
evalFgetpos(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const814*12c85518Srobert void StreamChecker::evalFgetpos(const FnDescription *Desc,
815*12c85518Srobert                                 const CallEvent &Call,
816*12c85518Srobert                                 CheckerContext &C) const {
817*12c85518Srobert   ProgramStateRef State = C.getState();
818*12c85518Srobert   SymbolRef Sym = getStreamArg(Desc, Call).getAsSymbol();
819*12c85518Srobert   if (!Sym)
820*12c85518Srobert     return;
821*12c85518Srobert 
822*12c85518Srobert   // Do not evaluate if stream is not found.
823*12c85518Srobert   if (!State->get<StreamMap>(Sym))
824*12c85518Srobert     return;
825*12c85518Srobert 
826*12c85518Srobert   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
827*12c85518Srobert   if (!CE)
828*12c85518Srobert     return;
829*12c85518Srobert 
830*12c85518Srobert   DefinedSVal RetVal = makeRetVal(C, CE);
831*12c85518Srobert   State = State->BindExpr(CE, C.getLocationContext(), RetVal);
832*12c85518Srobert   ProgramStateRef StateNotFailed, StateFailed;
833*12c85518Srobert   std::tie(StateFailed, StateNotFailed) =
834*12c85518Srobert       C.getConstraintManager().assumeDual(State, RetVal);
835*12c85518Srobert 
836*12c85518Srobert   // This function does not affect the stream state.
837*12c85518Srobert   // Still we add success and failure state with the appropriate return value.
838*12c85518Srobert   // StdLibraryFunctionsChecker can change these states (set the 'errno' state).
839*12c85518Srobert   C.addTransition(StateNotFailed);
840*12c85518Srobert   C.addTransition(StateFailed);
841*12c85518Srobert }
842*12c85518Srobert 
evalFsetpos(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const843*12c85518Srobert void StreamChecker::evalFsetpos(const FnDescription *Desc,
844*12c85518Srobert                                 const CallEvent &Call,
845*12c85518Srobert                                 CheckerContext &C) const {
846*12c85518Srobert   ProgramStateRef State = C.getState();
847*12c85518Srobert   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
848*12c85518Srobert   if (!StreamSym)
849*12c85518Srobert     return;
850*12c85518Srobert 
851*12c85518Srobert   const StreamState *SS = State->get<StreamMap>(StreamSym);
852*12c85518Srobert   if (!SS)
853*12c85518Srobert     return;
854*12c85518Srobert 
855*12c85518Srobert   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
856*12c85518Srobert   if (!CE)
857*12c85518Srobert     return;
858*12c85518Srobert 
859*12c85518Srobert   assertStreamStateOpened(SS);
860*12c85518Srobert 
861*12c85518Srobert   DefinedSVal RetVal = makeRetVal(C, CE);
862*12c85518Srobert   State = State->BindExpr(CE, C.getLocationContext(), RetVal);
863*12c85518Srobert   ProgramStateRef StateNotFailed, StateFailed;
864*12c85518Srobert   std::tie(StateFailed, StateNotFailed) =
865*12c85518Srobert       C.getConstraintManager().assumeDual(State, RetVal);
866*12c85518Srobert 
867*12c85518Srobert   StateNotFailed = StateNotFailed->set<StreamMap>(
868*12c85518Srobert       StreamSym, StreamState::getOpened(Desc, ErrorNone, false));
869*12c85518Srobert 
870*12c85518Srobert   // At failure ferror could be set.
871*12c85518Srobert   // The standards do not tell what happens with the file position at failure.
872*12c85518Srobert   // But we can assume that it is dangerous to make a next I/O operation after
873*12c85518Srobert   // the position was not set correctly (similar to 'fseek').
874*12c85518Srobert   StateFailed = StateFailed->set<StreamMap>(
875*12c85518Srobert       StreamSym, StreamState::getOpened(Desc, ErrorNone | ErrorFError, true));
876*12c85518Srobert 
877*12c85518Srobert   C.addTransition(StateNotFailed);
878*12c85518Srobert   C.addTransition(StateFailed);
879*12c85518Srobert }
880*12c85518Srobert 
evalFtell(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const881*12c85518Srobert void StreamChecker::evalFtell(const FnDescription *Desc, const CallEvent &Call,
882*12c85518Srobert                               CheckerContext &C) const {
883*12c85518Srobert   ProgramStateRef State = C.getState();
884*12c85518Srobert   SymbolRef Sym = getStreamArg(Desc, Call).getAsSymbol();
885*12c85518Srobert   if (!Sym)
886*12c85518Srobert     return;
887*12c85518Srobert 
888*12c85518Srobert   if (!State->get<StreamMap>(Sym))
889*12c85518Srobert     return;
890*12c85518Srobert 
891*12c85518Srobert   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
892*12c85518Srobert   if (!CE)
893*12c85518Srobert     return;
894*12c85518Srobert 
895*12c85518Srobert   SValBuilder &SVB = C.getSValBuilder();
896*12c85518Srobert   NonLoc RetVal = makeRetVal(C, CE).castAs<NonLoc>();
897*12c85518Srobert   ProgramStateRef StateNotFailed =
898*12c85518Srobert       State->BindExpr(CE, C.getLocationContext(), RetVal);
899*12c85518Srobert   auto Cond = SVB.evalBinOp(State, BO_GE, RetVal,
900*12c85518Srobert                             SVB.makeZeroVal(C.getASTContext().LongTy),
901*12c85518Srobert                             SVB.getConditionType())
902*12c85518Srobert                   .getAs<DefinedOrUnknownSVal>();
903*12c85518Srobert   if (!Cond)
904*12c85518Srobert     return;
905*12c85518Srobert   StateNotFailed = StateNotFailed->assume(*Cond, true);
906*12c85518Srobert   if (!StateNotFailed)
907*12c85518Srobert     return;
908*12c85518Srobert 
909*12c85518Srobert   ProgramStateRef StateFailed = State->BindExpr(
910*12c85518Srobert       CE, C.getLocationContext(), SVB.makeIntVal(-1, C.getASTContext().LongTy));
911*12c85518Srobert 
912*12c85518Srobert   C.addTransition(StateNotFailed);
913*12c85518Srobert   C.addTransition(StateFailed);
914*12c85518Srobert }
915*12c85518Srobert 
evalRewind(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const916*12c85518Srobert void StreamChecker::evalRewind(const FnDescription *Desc, const CallEvent &Call,
917*12c85518Srobert                                CheckerContext &C) const {
918*12c85518Srobert   ProgramStateRef State = C.getState();
919*12c85518Srobert   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
920*12c85518Srobert   if (!StreamSym)
921*12c85518Srobert     return;
922*12c85518Srobert 
923*12c85518Srobert   const StreamState *SS = State->get<StreamMap>(StreamSym);
924*12c85518Srobert   if (!SS)
925*12c85518Srobert     return;
926*12c85518Srobert 
927*12c85518Srobert   auto *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
928*12c85518Srobert   if (!CE)
929*12c85518Srobert     return;
930*12c85518Srobert 
931*12c85518Srobert   assertStreamStateOpened(SS);
932*12c85518Srobert 
933*12c85518Srobert   State = State->set<StreamMap>(StreamSym,
934*12c85518Srobert                                 StreamState::getOpened(Desc, ErrorNone, false));
935*12c85518Srobert 
936*12c85518Srobert   C.addTransition(State);
937*12c85518Srobert }
938*12c85518Srobert 
evalClearerr(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const939ec727ea7Spatrick void StreamChecker::evalClearerr(const FnDescription *Desc,
940ec727ea7Spatrick                                  const CallEvent &Call,
941ec727ea7Spatrick                                  CheckerContext &C) const {
942ec727ea7Spatrick   ProgramStateRef State = C.getState();
943ec727ea7Spatrick   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
944ec727ea7Spatrick   if (!StreamSym)
945ec727ea7Spatrick     return;
946ec727ea7Spatrick 
947ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(StreamSym);
948ec727ea7Spatrick   if (!SS)
949ec727ea7Spatrick     return;
950ec727ea7Spatrick 
951ec727ea7Spatrick   assertStreamStateOpened(SS);
952ec727ea7Spatrick 
953ec727ea7Spatrick   // FilePositionIndeterminate is not cleared.
954ec727ea7Spatrick   State = State->set<StreamMap>(
955ec727ea7Spatrick       StreamSym,
956ec727ea7Spatrick       StreamState::getOpened(Desc, ErrorNone, SS->FilePositionIndeterminate));
957ec727ea7Spatrick   C.addTransition(State);
958ec727ea7Spatrick }
959ec727ea7Spatrick 
evalFeofFerror(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C,const StreamErrorState & ErrorKind) const960ec727ea7Spatrick void StreamChecker::evalFeofFerror(const FnDescription *Desc,
961ec727ea7Spatrick                                    const CallEvent &Call, CheckerContext &C,
962ec727ea7Spatrick                                    const StreamErrorState &ErrorKind) const {
963ec727ea7Spatrick   ProgramStateRef State = C.getState();
964ec727ea7Spatrick   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
965ec727ea7Spatrick   if (!StreamSym)
966ec727ea7Spatrick     return;
967ec727ea7Spatrick 
968ec727ea7Spatrick   const CallExpr *CE = dyn_cast_or_null<CallExpr>(Call.getOriginExpr());
969ec727ea7Spatrick   if (!CE)
970ec727ea7Spatrick     return;
971ec727ea7Spatrick 
972ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(StreamSym);
973ec727ea7Spatrick   if (!SS)
974ec727ea7Spatrick     return;
975ec727ea7Spatrick 
976ec727ea7Spatrick   assertStreamStateOpened(SS);
977ec727ea7Spatrick 
978ec727ea7Spatrick   if (SS->ErrorState & ErrorKind) {
979ec727ea7Spatrick     // Execution path with error of ErrorKind.
980ec727ea7Spatrick     // Function returns true.
981ec727ea7Spatrick     // From now on it is the only one error state.
982ec727ea7Spatrick     ProgramStateRef TrueState = bindAndAssumeTrue(State, C, CE);
983ec727ea7Spatrick     C.addTransition(TrueState->set<StreamMap>(
984ec727ea7Spatrick         StreamSym, StreamState::getOpened(Desc, ErrorKind,
985ec727ea7Spatrick                                           SS->FilePositionIndeterminate &&
986ec727ea7Spatrick                                               !ErrorKind.isFEof())));
987ec727ea7Spatrick   }
988ec727ea7Spatrick   if (StreamErrorState NewES = SS->ErrorState & (~ErrorKind)) {
989ec727ea7Spatrick     // Execution path(s) with ErrorKind not set.
990ec727ea7Spatrick     // Function returns false.
991ec727ea7Spatrick     // New error state is everything before minus ErrorKind.
992ec727ea7Spatrick     ProgramStateRef FalseState = bindInt(0, State, C, CE);
993ec727ea7Spatrick     C.addTransition(FalseState->set<StreamMap>(
994ec727ea7Spatrick         StreamSym,
995ec727ea7Spatrick         StreamState::getOpened(
996ec727ea7Spatrick             Desc, NewES, SS->FilePositionIndeterminate && !NewES.isFEof())));
997ec727ea7Spatrick   }
998ec727ea7Spatrick }
999ec727ea7Spatrick 
preDefault(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C) const1000ec727ea7Spatrick void StreamChecker::preDefault(const FnDescription *Desc, const CallEvent &Call,
1001ec727ea7Spatrick                                CheckerContext &C) const {
1002ec727ea7Spatrick   ProgramStateRef State = C.getState();
1003ec727ea7Spatrick   SVal StreamVal = getStreamArg(Desc, Call);
1004a9ac8606Spatrick   State = ensureStreamNonNull(StreamVal, Call.getArgExpr(Desc->StreamArgNo), C,
1005a9ac8606Spatrick                               State);
1006ec727ea7Spatrick   if (!State)
1007ec727ea7Spatrick     return;
1008ec727ea7Spatrick   State = ensureStreamOpened(StreamVal, C, State);
1009ec727ea7Spatrick   if (!State)
1010ec727ea7Spatrick     return;
1011ec727ea7Spatrick 
1012ec727ea7Spatrick   C.addTransition(State);
1013ec727ea7Spatrick }
1014ec727ea7Spatrick 
evalSetFeofFerror(const FnDescription * Desc,const CallEvent & Call,CheckerContext & C,const StreamErrorState & ErrorKind) const1015ec727ea7Spatrick void StreamChecker::evalSetFeofFerror(const FnDescription *Desc,
1016ec727ea7Spatrick                                       const CallEvent &Call, CheckerContext &C,
1017ec727ea7Spatrick                                       const StreamErrorState &ErrorKind) const {
1018ec727ea7Spatrick   ProgramStateRef State = C.getState();
1019ec727ea7Spatrick   SymbolRef StreamSym = getStreamArg(Desc, Call).getAsSymbol();
1020ec727ea7Spatrick   assert(StreamSym && "Operation not permitted on non-symbolic stream value.");
1021ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(StreamSym);
1022ec727ea7Spatrick   assert(SS && "Stream should be tracked by the checker.");
1023ec727ea7Spatrick   State = State->set<StreamMap>(
1024ec727ea7Spatrick       StreamSym, StreamState::getOpened(SS->LastOperation, ErrorKind));
1025ec727ea7Spatrick   C.addTransition(State);
1026ec727ea7Spatrick }
1027ec727ea7Spatrick 
1028ec727ea7Spatrick ProgramStateRef
ensureStreamNonNull(SVal StreamVal,const Expr * StreamE,CheckerContext & C,ProgramStateRef State) const1029a9ac8606Spatrick StreamChecker::ensureStreamNonNull(SVal StreamVal, const Expr *StreamE,
1030a9ac8606Spatrick                                    CheckerContext &C,
1031ec727ea7Spatrick                                    ProgramStateRef State) const {
1032ec727ea7Spatrick   auto Stream = StreamVal.getAs<DefinedSVal>();
1033ec727ea7Spatrick   if (!Stream)
1034ec727ea7Spatrick     return State;
1035e5dd7070Spatrick 
1036e5dd7070Spatrick   ConstraintManager &CM = C.getConstraintManager();
1037ec727ea7Spatrick 
1038e5dd7070Spatrick   ProgramStateRef StateNotNull, StateNull;
1039ec727ea7Spatrick   std::tie(StateNotNull, StateNull) = CM.assumeDual(C.getState(), *Stream);
1040e5dd7070Spatrick 
1041e5dd7070Spatrick   if (!StateNotNull && StateNull) {
1042*12c85518Srobert     // Stream argument is NULL, stop analysis on this path.
1043*12c85518Srobert     // This case should occur only if StdLibraryFunctionsChecker (or ModelPOSIX
1044*12c85518Srobert     // option of it) is not turned on, otherwise that checker ensures non-null
1045*12c85518Srobert     // argument.
1046*12c85518Srobert     C.generateSink(StateNull, C.getPredecessor());
1047ec727ea7Spatrick     return nullptr;
1048e5dd7070Spatrick   }
1049e5dd7070Spatrick 
1050ec727ea7Spatrick   return StateNotNull;
1051e5dd7070Spatrick }
1052e5dd7070Spatrick 
ensureStreamOpened(SVal StreamVal,CheckerContext & C,ProgramStateRef State) const1053ec727ea7Spatrick ProgramStateRef StreamChecker::ensureStreamOpened(SVal StreamVal,
1054ec727ea7Spatrick                                                   CheckerContext &C,
1055ec727ea7Spatrick                                                   ProgramStateRef State) const {
1056ec727ea7Spatrick   SymbolRef Sym = StreamVal.getAsSymbol();
1057ec727ea7Spatrick   if (!Sym)
1058ec727ea7Spatrick     return State;
1059ec727ea7Spatrick 
1060ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(Sym);
1061ec727ea7Spatrick   if (!SS)
1062ec727ea7Spatrick     return State;
1063ec727ea7Spatrick 
1064ec727ea7Spatrick   if (SS->isClosed()) {
1065ec727ea7Spatrick     // Using a stream pointer after 'fclose' causes undefined behavior
1066ec727ea7Spatrick     // according to cppreference.com .
1067ec727ea7Spatrick     ExplodedNode *N = C.generateErrorNode();
1068ec727ea7Spatrick     if (N) {
1069ec727ea7Spatrick       C.emitReport(std::make_unique<PathSensitiveBugReport>(
1070ec727ea7Spatrick           BT_UseAfterClose,
1071ec727ea7Spatrick           "Stream might be already closed. Causes undefined behaviour.", N));
1072ec727ea7Spatrick       return nullptr;
1073e5dd7070Spatrick     }
1074e5dd7070Spatrick 
1075ec727ea7Spatrick     return State;
1076ec727ea7Spatrick   }
1077ec727ea7Spatrick 
1078ec727ea7Spatrick   if (SS->isOpenFailed()) {
1079ec727ea7Spatrick     // Using a stream that has failed to open is likely to cause problems.
1080ec727ea7Spatrick     // This should usually not occur because stream pointer is NULL.
1081ec727ea7Spatrick     // But freopen can cause a state when stream pointer remains non-null but
1082ec727ea7Spatrick     // failed to open.
1083ec727ea7Spatrick     ExplodedNode *N = C.generateErrorNode();
1084ec727ea7Spatrick     if (N) {
1085ec727ea7Spatrick       C.emitReport(std::make_unique<PathSensitiveBugReport>(
1086ec727ea7Spatrick           BT_UseAfterOpenFailed,
1087ec727ea7Spatrick           "Stream might be invalid after "
1088ec727ea7Spatrick           "(re-)opening it has failed. "
1089ec727ea7Spatrick           "Can cause undefined behaviour.",
1090ec727ea7Spatrick           N));
1091ec727ea7Spatrick       return nullptr;
1092ec727ea7Spatrick     }
1093ec727ea7Spatrick     return State;
1094ec727ea7Spatrick   }
1095ec727ea7Spatrick 
1096ec727ea7Spatrick   return State;
1097ec727ea7Spatrick }
1098ec727ea7Spatrick 
ensureNoFilePositionIndeterminate(SVal StreamVal,CheckerContext & C,ProgramStateRef State) const1099ec727ea7Spatrick ProgramStateRef StreamChecker::ensureNoFilePositionIndeterminate(
1100ec727ea7Spatrick     SVal StreamVal, CheckerContext &C, ProgramStateRef State) const {
1101ec727ea7Spatrick   static const char *BugMessage =
1102ec727ea7Spatrick       "File position of the stream might be 'indeterminate' "
1103ec727ea7Spatrick       "after a failed operation. "
1104ec727ea7Spatrick       "Can cause undefined behavior.";
1105ec727ea7Spatrick 
1106ec727ea7Spatrick   SymbolRef Sym = StreamVal.getAsSymbol();
1107ec727ea7Spatrick   if (!Sym)
1108ec727ea7Spatrick     return State;
1109ec727ea7Spatrick 
1110ec727ea7Spatrick   const StreamState *SS = State->get<StreamMap>(Sym);
1111ec727ea7Spatrick   if (!SS)
1112ec727ea7Spatrick     return State;
1113ec727ea7Spatrick 
1114ec727ea7Spatrick   assert(SS->isOpened() && "First ensure that stream is opened.");
1115ec727ea7Spatrick 
1116ec727ea7Spatrick   if (SS->FilePositionIndeterminate) {
1117ec727ea7Spatrick     if (SS->ErrorState & ErrorFEof) {
1118ec727ea7Spatrick       // The error is unknown but may be FEOF.
1119ec727ea7Spatrick       // Continue analysis with the FEOF error state.
1120ec727ea7Spatrick       // Report warning because the other possible error states.
1121ec727ea7Spatrick       ExplodedNode *N = C.generateNonFatalErrorNode(State);
1122ec727ea7Spatrick       if (!N)
1123ec727ea7Spatrick         return nullptr;
1124ec727ea7Spatrick 
1125ec727ea7Spatrick       C.emitReport(std::make_unique<PathSensitiveBugReport>(
1126ec727ea7Spatrick           BT_IndeterminatePosition, BugMessage, N));
1127ec727ea7Spatrick       return State->set<StreamMap>(
1128ec727ea7Spatrick           Sym, StreamState::getOpened(SS->LastOperation, ErrorFEof, false));
1129ec727ea7Spatrick     }
1130ec727ea7Spatrick 
1131ec727ea7Spatrick     // Known or unknown error state without FEOF possible.
1132ec727ea7Spatrick     // Stop analysis, report error.
1133ec727ea7Spatrick     ExplodedNode *N = C.generateErrorNode(State);
1134ec727ea7Spatrick     if (N)
1135ec727ea7Spatrick       C.emitReport(std::make_unique<PathSensitiveBugReport>(
1136ec727ea7Spatrick           BT_IndeterminatePosition, BugMessage, N));
1137ec727ea7Spatrick 
1138ec727ea7Spatrick     return nullptr;
1139ec727ea7Spatrick   }
1140ec727ea7Spatrick 
1141ec727ea7Spatrick   return State;
1142ec727ea7Spatrick }
1143ec727ea7Spatrick 
1144ec727ea7Spatrick ProgramStateRef
ensureFseekWhenceCorrect(SVal WhenceVal,CheckerContext & C,ProgramStateRef State) const1145ec727ea7Spatrick StreamChecker::ensureFseekWhenceCorrect(SVal WhenceVal, CheckerContext &C,
1146ec727ea7Spatrick                                         ProgramStateRef State) const {
1147*12c85518Srobert   std::optional<nonloc::ConcreteInt> CI =
1148*12c85518Srobert       WhenceVal.getAs<nonloc::ConcreteInt>();
1149e5dd7070Spatrick   if (!CI)
1150ec727ea7Spatrick     return State;
1151e5dd7070Spatrick 
1152e5dd7070Spatrick   int64_t X = CI->getValue().getSExtValue();
1153e5dd7070Spatrick   if (X >= 0 && X <= 2)
1154ec727ea7Spatrick     return State;
1155e5dd7070Spatrick 
1156e5dd7070Spatrick   if (ExplodedNode *N = C.generateNonFatalErrorNode(State)) {
1157ec727ea7Spatrick     C.emitReport(std::make_unique<PathSensitiveBugReport>(
1158ec727ea7Spatrick         BT_IllegalWhence,
1159e5dd7070Spatrick         "The whence argument to fseek() should be "
1160ec727ea7Spatrick         "SEEK_SET, SEEK_END, or SEEK_CUR.",
1161ec727ea7Spatrick         N));
1162ec727ea7Spatrick     return nullptr;
1163ec727ea7Spatrick   }
1164ec727ea7Spatrick 
1165ec727ea7Spatrick   return State;
1166ec727ea7Spatrick }
1167ec727ea7Spatrick 
reportFEofWarning(SymbolRef StreamSym,CheckerContext & C,ProgramStateRef State) const1168a9ac8606Spatrick void StreamChecker::reportFEofWarning(SymbolRef StreamSym, CheckerContext &C,
1169ec727ea7Spatrick                                       ProgramStateRef State) const {
1170ec727ea7Spatrick   if (ExplodedNode *N = C.generateNonFatalErrorNode(State)) {
1171a9ac8606Spatrick     auto R = std::make_unique<PathSensitiveBugReport>(
1172ec727ea7Spatrick         BT_StreamEof,
1173ec727ea7Spatrick         "Read function called when stream is in EOF state. "
1174ec727ea7Spatrick         "Function has no effect.",
1175a9ac8606Spatrick         N);
1176a9ac8606Spatrick     R->markInteresting(StreamSym);
1177a9ac8606Spatrick     C.emitReport(std::move(R));
1178ec727ea7Spatrick     return;
1179e5dd7070Spatrick   }
1180ec727ea7Spatrick   C.addTransition(State);
1181e5dd7070Spatrick }
1182e5dd7070Spatrick 
1183a9ac8606Spatrick ExplodedNode *
reportLeaks(const SmallVector<SymbolRef,2> & LeakedSyms,CheckerContext & C,ExplodedNode * Pred) const1184a9ac8606Spatrick StreamChecker::reportLeaks(const SmallVector<SymbolRef, 2> &LeakedSyms,
1185a9ac8606Spatrick                            CheckerContext &C, ExplodedNode *Pred) const {
1186a9ac8606Spatrick   ExplodedNode *Err = C.generateNonFatalErrorNode(C.getState(), Pred);
1187a9ac8606Spatrick   if (!Err)
1188a9ac8606Spatrick     return Pred;
1189e5dd7070Spatrick 
1190a9ac8606Spatrick   for (SymbolRef LeakSym : LeakedSyms) {
1191ec727ea7Spatrick     // Resource leaks can result in multiple warning that describe the same kind
1192ec727ea7Spatrick     // of programming error:
1193ec727ea7Spatrick     //  void f() {
1194ec727ea7Spatrick     //    FILE *F = fopen("a.txt");
1195ec727ea7Spatrick     //    if (rand()) // state split
1196ec727ea7Spatrick     //      return; // warning
1197ec727ea7Spatrick     //  } // warning
1198ec727ea7Spatrick     // While this isn't necessarily true (leaking the same stream could result
1199ec727ea7Spatrick     // from a different kinds of errors), the reduction in redundant reports
1200ec727ea7Spatrick     // makes this a worthwhile heuristic.
1201ec727ea7Spatrick     // FIXME: Add a checker option to turn this uniqueing feature off.
1202a9ac8606Spatrick     const ExplodedNode *StreamOpenNode = getAcquisitionSite(Err, LeakSym, C);
1203ec727ea7Spatrick     assert(StreamOpenNode && "Could not find place of stream opening.");
1204ec727ea7Spatrick     PathDiagnosticLocation LocUsedForUniqueing =
1205ec727ea7Spatrick         PathDiagnosticLocation::createBegin(
1206ec727ea7Spatrick             StreamOpenNode->getStmtForDiagnostics(), C.getSourceManager(),
1207ec727ea7Spatrick             StreamOpenNode->getLocationContext());
1208ec727ea7Spatrick 
1209ec727ea7Spatrick     std::unique_ptr<PathSensitiveBugReport> R =
1210ec727ea7Spatrick         std::make_unique<PathSensitiveBugReport>(
1211ec727ea7Spatrick             BT_ResourceLeak,
1212a9ac8606Spatrick             "Opened stream never closed. Potential resource leak.", Err,
1213ec727ea7Spatrick             LocUsedForUniqueing,
1214ec727ea7Spatrick             StreamOpenNode->getLocationContext()->getDecl());
1215a9ac8606Spatrick     R->markInteresting(LeakSym);
1216ec727ea7Spatrick     C.emitReport(std::move(R));
1217e5dd7070Spatrick   }
1218a9ac8606Spatrick 
1219a9ac8606Spatrick   return Err;
1220a9ac8606Spatrick }
1221a9ac8606Spatrick 
checkDeadSymbols(SymbolReaper & SymReaper,CheckerContext & C) const1222a9ac8606Spatrick void StreamChecker::checkDeadSymbols(SymbolReaper &SymReaper,
1223a9ac8606Spatrick                                      CheckerContext &C) const {
1224a9ac8606Spatrick   ProgramStateRef State = C.getState();
1225a9ac8606Spatrick 
1226a9ac8606Spatrick   llvm::SmallVector<SymbolRef, 2> LeakedSyms;
1227a9ac8606Spatrick 
1228a9ac8606Spatrick   const StreamMapTy &Map = State->get<StreamMap>();
1229a9ac8606Spatrick   for (const auto &I : Map) {
1230a9ac8606Spatrick     SymbolRef Sym = I.first;
1231a9ac8606Spatrick     const StreamState &SS = I.second;
1232a9ac8606Spatrick     if (!SymReaper.isDead(Sym))
1233a9ac8606Spatrick       continue;
1234a9ac8606Spatrick     if (SS.isOpened())
1235a9ac8606Spatrick       LeakedSyms.push_back(Sym);
1236a9ac8606Spatrick     State = State->remove<StreamMap>(Sym);
1237a9ac8606Spatrick   }
1238a9ac8606Spatrick 
1239a9ac8606Spatrick   ExplodedNode *N = C.getPredecessor();
1240a9ac8606Spatrick   if (!LeakedSyms.empty())
1241a9ac8606Spatrick     N = reportLeaks(LeakedSyms, C, N);
1242a9ac8606Spatrick 
1243a9ac8606Spatrick   C.addTransition(State, N);
1244e5dd7070Spatrick }
1245e5dd7070Spatrick 
checkPointerEscape(ProgramStateRef State,const InvalidatedSymbols & Escaped,const CallEvent * Call,PointerEscapeKind Kind) const1246ec727ea7Spatrick ProgramStateRef StreamChecker::checkPointerEscape(
1247ec727ea7Spatrick     ProgramStateRef State, const InvalidatedSymbols &Escaped,
1248ec727ea7Spatrick     const CallEvent *Call, PointerEscapeKind Kind) const {
1249ec727ea7Spatrick   // Check for file-handling system call that is not handled by the checker.
1250ec727ea7Spatrick   // FIXME: The checker should be updated to handle all system calls that take
1251ec727ea7Spatrick   // 'FILE*' argument. These are now ignored.
1252ec727ea7Spatrick   if (Kind == PSK_DirectEscapeOnCall && Call->isInSystemHeader())
1253ec727ea7Spatrick     return State;
1254ec727ea7Spatrick 
1255ec727ea7Spatrick   for (SymbolRef Sym : Escaped) {
1256ec727ea7Spatrick     // The symbol escaped.
1257ec727ea7Spatrick     // From now the stream can be manipulated in unknown way to the checker,
1258ec727ea7Spatrick     // it is not possible to handle it any more.
1259ec727ea7Spatrick     // Optimistically, assume that the corresponding file handle will be closed
1260ec727ea7Spatrick     // somewhere else.
1261ec727ea7Spatrick     // Remove symbol from state so the following stream calls on this symbol are
1262ec727ea7Spatrick     // not handled by the checker.
1263ec727ea7Spatrick     State = State->remove<StreamMap>(Sym);
1264ec727ea7Spatrick   }
1265ec727ea7Spatrick   return State;
1266e5dd7070Spatrick }
1267e5dd7070Spatrick 
1268a9ac8606Spatrick //===----------------------------------------------------------------------===//
1269a9ac8606Spatrick // Checker registration.
1270a9ac8606Spatrick //===----------------------------------------------------------------------===//
1271a9ac8606Spatrick 
registerStreamChecker(CheckerManager & Mgr)1272ec727ea7Spatrick void ento::registerStreamChecker(CheckerManager &Mgr) {
1273ec727ea7Spatrick   Mgr.registerChecker<StreamChecker>();
1274ec727ea7Spatrick }
1275ec727ea7Spatrick 
shouldRegisterStreamChecker(const CheckerManager & Mgr)1276ec727ea7Spatrick bool ento::shouldRegisterStreamChecker(const CheckerManager &Mgr) {
1277ec727ea7Spatrick   return true;
1278ec727ea7Spatrick }
1279ec727ea7Spatrick 
registerStreamTesterChecker(CheckerManager & Mgr)1280ec727ea7Spatrick void ento::registerStreamTesterChecker(CheckerManager &Mgr) {
1281ec727ea7Spatrick   auto *Checker = Mgr.getChecker<StreamChecker>();
1282ec727ea7Spatrick   Checker->TestMode = true;
1283ec727ea7Spatrick }
1284ec727ea7Spatrick 
shouldRegisterStreamTesterChecker(const CheckerManager & Mgr)1285ec727ea7Spatrick bool ento::shouldRegisterStreamTesterChecker(const CheckerManager &Mgr) {
1286e5dd7070Spatrick   return true;
1287e5dd7070Spatrick }
1288