365765 – [dstore][multithread]client environment cause harm to singe-process server

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 365765 - [dstore][multithread]client environment cause harm to singe-process server

Summary: [dstore][multithread]client environment cause harm to singe-process server

Status:	RESOLVED FIXED

Alias:	None

Product:	Target Management
Classification:	Tools
Component:	RSE (show other bugs)
Version:	unspecified
Hardware:	Other All

Importance:	P3 normal (vote)
Target Milestone:	3.4 M4
Assignee:	David McKnight
QA Contact:	Martin Oberhuber

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	365816
	Show dependency tree

Reported:	2011-12-06 11:32 EST by Noriaki Takatsu
Modified:	2012-06-12 16:23 EDT (History)
CC List:	2 users (show)

See Also:

Attachments
Patch for protecting the server shell encoding from client setting (1.64 KB, patch) 2011-12-06 11:45 EST, Noriaki Takatsu	mober.at+eclipse: iplog+	Details \| Diff
updated patch with copyright statement (2.26 KB, patch) 2011-12-06 12:29 EST, David McKnight	mober.at+eclipse: iplog- dmcknigh: review+	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Noriaki Takatsu

2011-12-06 11:32:29 EST

In multithread environment, a single-process server is shared among clients.
The client setting for shell encoding is propagated to the server setting and the server setting causes damage to other clients.

Comment 1 Noriaki Takatsu

2011-12-06 11:45:32 EST

Created attachment 208003 [details]
Patch for protecting the server shell encoding from client setting

Comment 2 David McKnight

2011-12-06 11:52:45 EST

Thanks for the patch, Noriaki.  Could you add a legal disclaimer?  Does this need to be backported?

Comment 3 Noriaki Takatsu

2011-12-06 12:20:42 EST

Legal Message:
 I, {Noriaki Takatsu}, declare that I developed the attached code from scratch, 
 without referencing any 3rd party materials except material licensed under 
 the EPL. 
 I am authorized by my employer to make this contribution under the EPL.

Comment 4 David McKnight

2011-12-06 12:29:35 EST

Created attachment 208007 [details]
updated patch with copyright statement

Comment 5 David McKnight

2011-12-06 12:31:52 EST

I've committed the change to the HEAD stream.  I'll create backport(s) depending on which release(s) this is needed for.

Comment 6 Onno Van den Troost

2011-12-06 16:20:03 EST

We need a backport to v3.2.1