search

lightblue-platform / lightblue-migrator

GNU General Public License v3.0
3 stars 13 forks source link

CID 114694: (LOCK_EVASION) #374

Open jewzaam opened 8 years ago

jewzaam commented 8 years ago

https://scan.coverity.com/projects/lightblue-platform-lightblue

*** CID 114694:    (LOCK_EVASION)
/lightblue-migrator/migrator/src/main/java/com/redhat/lightblue/migrator/MigratorController.java: 168 in com.redhat.lightblue.migrator.MigratorController.run()()
162                                     MigrationConfiguration x = reloadMigrationConfiguration();
163                                     if (x == null) {
164                                         // Terminate
165                                         LOGGER.debug("Controller {} terminating",migrationConfiguration.getConfigurationName());
166                                         stopped=true;
167                                     } else {
>>>     CID 114694:    (LOCK_EVASION)
>>>     The modification of "migrationConfiguration" can race with the unguarded check of "stopped".
168                                         migrationConfiguration = x;
169                                     }
170                                 } catch (Exception e) {
171                                     LOGGER.error("Cannot refresh configuration", e);
172                                 }
173                             }
/lightblue-migrator/migrator/src/main/java/com/redhat/lightblue/migrator/MigratorController.java: 168 in com.redhat.lightblue.migrator.MigratorController.run()()
162                                     MigrationConfiguration x = reloadMigrationConfiguration();
163                                     if (x == null) {
164                                         // Terminate
165                                         LOGGER.debug("Controller {} terminating",migrationConfiguration.getConfigurationName());
166                                         stopped=true;
167                                     } else {
>>>     CID 114694:    (LOCK_EVASION)
>>>     The modification of "migrationConfiguration" can race with the unguarded check of "stopped".
168                                         migrationConfiguration = x;
169                                     }
170                                 } catch (Exception e) {
171                                     LOGGER.error("Cannot refresh configuration", e);
172                                 }
173                             }