You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by "Stefan Seelmann (JIRA)" <ji...@apache.org> on 2016/03/15 21:36:33 UTC

[jira] [Updated] (DIRSTUDIO-1100) Race condition.

     [ https://issues.apache.org/jira/browse/DIRSTUDIO-1100?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Stefan Seelmann updated DIRSTUDIO-1100:
---------------------------------------
    Fix Version/s: 2.0.0-M11

> Race condition.
> ---------------
>
>                 Key: DIRSTUDIO-1100
>                 URL: https://issues.apache.org/jira/browse/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
>             Fix For: 2.0.0-M11
>
>
> Hello,
> 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):
> Firstly:
> -> EntryEditorUtils.ensureAttributesInitialized(entry);
>   -> StudioBrowserJob runs InitializeAttributesRunnable in a thread
>     -> InitializeAttributesRunnable.run
>       -> initializeAttributes( entry, monitor ) -> initializeAttributes( entry, returningAttributes, true, monitor )
>           -> entry.deleteAttribute( oldAttribute ) ->   ai.attributeMap.remove(Strings.toLowerCase(oidString));
> Secondly:
> -> 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
(v6.3.4#6332)