Modify

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11250 closed defect (fixed)

[Patch] User#userMap is not synchronized

Reported by: michael2402 Owned by: team
Priority: normal Milestone: 15.03
Component: Core Version:
Keywords: Cc:

Description

This should not trigger often, but if it does, it might screw things up a lot (since HashMap is not synchronized itself).

Attachments (1)

0001-Made-user-map-synchronized.patch (3.5 KB) - added by michael2402 5 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 5 years ago by bastiK

Yes, this looks unsafe. Why not simply add a synchronized modifier for the methods?

comment:2 Changed 5 years ago by michael2402

Using a plain synchronized would also work in this case.

comment:3 Changed 5 years ago by bastiK

Could you update the patch using plain synchronized?

Changed 5 years ago by michael2402

comment:4 Changed 5 years ago by michael2402

I updated the patch.

comment:5 Changed 5 years ago by bastiK

Resolution: fixed
Status: newclosed

In 8155/josm:

applied #11250 - User#userMap is not synchronized (patch by michael2402)

comment:6 Changed 5 years ago by bastiK

Thanks!

comment:7 Changed 5 years ago by Don-vip

Milestone: 15.03

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.