Re: New Defects reported by Coverity Scan for ceph

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The first exception should be shadowed?

And the second exception seemed strange, because other tests are follow this way

On Sat, Jan 10, 2015 at 10:36 PM,  <scan-admin@xxxxxxxxxxxx> wrote:
>
> Hi,
>
> Please find the latest report on new defect(s) introduced to ceph found with Coverity Scan.
>
> 2 new defect(s) introduced to ceph found with Coverity Scan.
>
>
> New defect(s) Reported-by: Coverity Scan
> Showing 2 of 2 defect(s)
>
>
> ** CID 1260210:  Resource leak  (RESOURCE_LEAK)
> /test/msgr/test_msgr.cc: 537 in MessengerTest_ClientStandbyTest_Test::TestBody()()
>
> ** CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
> /test/msgr/test_msgr.cc: 579 in main()
> /test/msgr/test_msgr.cc: 579 in main()
> /test/msgr/test_msgr.cc: 579 in main()
> /test/msgr/test_msgr.cc: 579 in main()
> /test/msgr/test_msgr.cc: 579 in main()
> /test/msgr/test_msgr.cc: 579 in main()
>
>
> ________________________________________________________________________________________________________
> *** CID 1260210:  Resource leak  (RESOURCE_LEAK)
> /test/msgr/test_msgr.cc: 537 in MessengerTest_ClientStandbyTest_Test::TestBody()()
> 531       usleep(300*1000);
> 532       // client should be standby, so we use original connection
> 533       {
> 534         m = new MPing();
> 535         conn->send_keepalive();
> 536         CHECK_AND_WAIT_TRUE(conn->is_connected());
>>>>     CID 1260210:  Resource leak  (RESOURCE_LEAK)
>>>>     Variable "m" going out of scope leaks the storage it points to.
> 537         ASSERT_TRUE(conn->is_connected());
> 538         ASSERT_EQ(conn->send_message(m), 0);
> 539         Mutex::Locker l(cli_dispatcher.lock);
> 540         while (!cli_dispatcher.got_new)
> 541           cli_dispatcher.cond.Wait(cli_dispatcher.lock);
> 542         cli_dispatcher.got_new = false;
>
> ________________________________________________________________________________________________________
> *** CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
> /test/msgr/test_msgr.cc: 579 in main()
> 573     // must be defined). This dummy test keeps gtest_main linked in.
> 574     TEST(DummyTest, ValueParameterizedTestsAreNotSupportedOnThisPlatform) {}
> 575
> 576     #endif
> 577
> 578
>>>>     CID 1260212:  Uncaught exception  (UNCAUGHT_EXCEPT)
>>>>     In function "main(int, char **)" an exception of type "ceph::FailedAssertion" is thrown and never caught.
> 579     int main(int argc, char **argv) {
> 580       vector<const char*> args;
> 581       argv_to_vec(argc, (const char **)argv, args);
> 582
> 583       global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY, 0);
> 584       g_ceph_context->_conf->set_val("auth_cluster_required", "none");
>
>
> ________________________________________________________________________________________________________
> To view the defects in Coverity Scan visit, http://scan.coverity.com/projects/25?tab=overview
>
> To manage Coverity Scan email notifications for "ceph-devel@xxxxxxxxxxxxxxx", click http://scan.coverity.com/subscriptions/edit?email=ceph-devel%40vger.kernel.org&token=018084d671e3cc89d00dd2ccb7eb849c .
>
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Best Regards,

Wheat
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux