asfimport

Results 862 comments of asfimport

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@pmouawad ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c2)): Thanks for report, I was wondering why your tool does not also report line 456: return new PropertyIteratorImpl(propMap.values()); Did you report all findings of your tool...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

**Bob Atkey** ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c3)): > Thanks for report, I was wondering why your tool does not also report line 456: > return new PropertyIteratorImpl(propMap.values()); Actually, our tool doesn't report...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@ham1 ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c4)): Created PR for the example identified in this issue: https://github.com/apache/jmeter/pull/260 Are there any more that the tool found (in the latest code) which should be addressed?

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@vlsi ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c5)): As far as I can see, the syncrhonizedMap was added in attempt to fix https://github.com/apache/jmeter/issues/1023 The latest reply suggests that the synchronization did not help. So...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@pmouawad ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c6)): (In reply to Vladimir Sitnikov from comment 5) > As far as I can see, the syncrhonizedMap was added in attempt to fix > https://github.com/apache/jmeter/issues/1023 >...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@ham1 ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c7)): (In reply to Vladimir Sitnikov from comment 5) > As far as I can see, the syncrhonizedMap was added in attempt to fix > https://github.com/apache/jmeter/issues/1023 >...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

**TaoLu** ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c8)): public void recoverRunningVersion() { Iterator iter = this.propMap.entrySet().iterator(); while(iter.hasNext()) { Entry<String, JMeterProperty> entry = (Entry)iter.next(); JMeterProperty prop = (JMeterProperty)entry.getValue(); if (this.isTemporary(prop)) { iter.remove(); this.clearTemporary(prop); } else...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

**TaoLu** ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c9)): For the 10 slave Linux machines used in the test environment, each 10 thread is 100 concurrent, and most of them report an error.ConcurrentModificationException, The sampler...

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@vlsi ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c10)): @TaoLu, could you please share the script that reproduces the issue?

Iteration over a view on a synchronized collection without obtaining a lock on the collection

@FSchumacher ([migrated from Bugzilla](https://bz.apache.org/bugzilla//show_bug.cgi?id=55827&redirect=false#c11)): I think we have to use a lock around the clean-up of the temporary properties like Graham did. With the attached patch I can reproduce an...