Committed by
Gerrit Code Review
Avoid concurrent modification exception by iterating over a immutable collection
Change-Id: Icfffa1347071eb298a25b11ee0511f36002b0675
Showing
1 changed file
with
1 additions
and
2 deletions
| ... | @@ -47,7 +47,6 @@ import org.apache.felix.scr.annotations.Reference; | ... | @@ -47,7 +47,6 @@ import org.apache.felix.scr.annotations.Reference; |
| 47 | import org.apache.felix.scr.annotations.ReferenceCardinality; | 47 | import org.apache.felix.scr.annotations.ReferenceCardinality; |
| 48 | import org.apache.felix.scr.annotations.ReferencePolicy; | 48 | import org.apache.felix.scr.annotations.ReferencePolicy; |
| 49 | import org.apache.felix.scr.annotations.Service; | 49 | import org.apache.felix.scr.annotations.Service; |
| 50 | - | ||
| 51 | import org.onosproject.app.ApplicationEvent; | 50 | import org.onosproject.app.ApplicationEvent; |
| 52 | import org.onosproject.app.ApplicationListener; | 51 | import org.onosproject.app.ApplicationListener; |
| 53 | import org.onosproject.app.ApplicationService; | 52 | import org.onosproject.app.ApplicationService; |
| ... | @@ -252,7 +251,7 @@ public class DatabaseManager implements StorageService, StorageAdminService { | ... | @@ -252,7 +251,7 @@ public class DatabaseManager implements StorageService, StorageAdminService { |
| 252 | log.info("Successfully closed databases."); | 251 | log.info("Successfully closed databases."); |
| 253 | } | 252 | } |
| 254 | }); | 253 | }); |
| 255 | - maps.values().forEach(this::unregisterMap); | 254 | + ImmutableList.copyOf(maps.values()).forEach(this::unregisterMap); |
| 256 | if (applicationService != null) { | 255 | if (applicationService != null) { |
| 257 | applicationService.removeListener(appListener); | 256 | applicationService.removeListener(appListener); |
| 258 | } | 257 | } | ... | ... |
-
Please register or login to post a comment