directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Václav Havlík (JIRA) <>
Subject [jira] [Created] (DIRSTUDIO-1100) Race condition.
Date Tue, 15 Mar 2016 09:48:33 GMT
Václav Havlík created DIRSTUDIO-1100:

             Summary: Race condition.
                 Key: DIRSTUDIO-1100
             Project: Directory Studio
          Issue Type: Bug
          Components: studio-ldapbrowser
    Affects Versions: 2.0.0-M10 (2.0.0.v20151221-M10)
         Environment: x86_64, Linux Debian 8, GTK
            Reporter: Václav Havlík
            Priority: Minor

perhaps I am wrong, but I suspect myself of seeing a bug in Apache Directory Studio 2.0.0.v20151221-M10.

In method EntryEditorManager.updateAutoSaveSharedReferenceCopy (runs in main thread):

-> EntryEditorUtils.ensureAttributesInitialized(entry);
  -> StudioBrowserJob runs InitializeAttributesRunnable in a thread
      -> initializeAttributes( entry, monitor ) -> initializeAttributes( entry, returningAttributes,
true, monitor )
          -> entry.deleteAttribute( oldAttribute ) ->   ai.attributeMap.remove(Strings.toLowerCase(oidString));

-> new CompoundModification().replaceAttributes(entry, workingCopy, this);
    -> fromEntry.getAttributes() -> ai.attributeMap.values()

Thus, the 1st thread modifies ai.attributeMap and at the same time the 2nd thread reads it.
Only initializeAttributes is synchronized, but on its class object.

Is it not a race condition?

Thank you. With regards Vaclav Havlik.

This message was sent by Atlassian JIRA

View raw message