[jira] [Created] (MESOS-1353) Operands don't affect result in mesos_containerizer.cpp

["Niklas Quarfot Nielsen (JIRA)" <ji...@apache.org>]

Niklas Quarfot Nielsen created MESOS-1353:
---------------------------------------------

             Summary: Operands don't affect result in mesos_containerizer.cpp
                 Key: MESOS-1353
                 URL: https://issues.apache.org/jira/browse/MESOS-1353
             Project: Mesos
          Issue Type: Technical task
            Reporter: Niklas Quarfot Nielsen
            Priority: Minor


May be a false positive - should be investigated.
________________________________________________________________________________________________________
*** CID 1213887:  Operands don't affect result  (CONSTANT_EXPRESSION_RESULT)
/src/slave/containerizer/mesos_containerizer.cpp: 416 in mesos::internal::slave::execute(const mesos::CommandInfo &, const std::basic_string<char, std::char_traits<char>, std::allocator<char>>&, const Option<std::basic_string<char, std::char_traits<char>, std::allocator<char>>> &, const std::map<std::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::less<std::basic_string<char, std::char_traits<char>, std::allocator<char>>>, std::allocator<std::pair<const std::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::basic_string<char, std::char_traits<char>, std::allocator<char>>>>> &, bool, int, int, const std::list<Option<mesos::CommandInfo>, std::allocator<Option<mesos::CommandInfo>>> &)()
410         if (chown.isError()) {
411           ABORT("Failed to chown work directory");
412         }
413       }
414
415       // Enter working directory.
>>>     CID 1213887:  Operands don't affect result  (CONSTANT_EXPRESSION_RESULT)
>>>     "chdir(directory) < 0" is always false regardless of the values of its operands. This occurs as the logical operand of if.
416       if (os::chdir(directory) < 0) {
417         ABORT("Failed to chdir into work directory");
418       }
419
420       // Redirect output to files in working dir if required. We append because
421       // others (e.g., mesos-fetcher) may have already logged to the files.




--
This message was sent by Atlassian JIRA
(v6.2#6252)