<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv=Content-Type content="text/html; charset=iso-8859-1">
<meta name=Generator content="Microsoft Word 12 (filtered medium)">
<style>
<!--
/* Font Definitions */
@font-face
{font-family:Wingdings;
panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;}
@page Section1
{size:612.0pt 792.0pt;
margin:2.0cm 42.5pt 2.0cm 3.0cm;}
div.Section1
{page:Section1;}
-->
</style>
<!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang=RU link=blue vlink=purple>
<div class=Section1>
<p class=MsoNormal><span lang=EN-US> ACE VERSION: 5.6.1 <o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> AREA/CLASS/EXAMPLE AFFECTED: ACE_Hash_Multi_Map_Manager<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> DOES THE PROBLEM AFFECT:<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> EXECUTION? - YES<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> SYNOPSIS:<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>Redundant copying in bind_i method<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> DESCRIPTION:<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>The code below presented in ACE_Hash_Multi_Map_Manager
at least since 5.5.3.<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US>template <class EXT_ID, class INT_ID,
class HASH_KEY, class COMPARE_KEYS, class ACE_LOCK> int<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>ACE_Hash_Multi_Map_Manager<EXT_ID,
INT_ID, HASH_KEY, COMPARE_KEYS, ACE_LOCK>::bind_i (const EXT_ID &ext_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> const
INT_ID &int_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
ACE_Hash_Multi_Map_Entry<EXT_ID, INT_ID> *&entry)<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>{<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> size_t loc;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> int result = this->shared_find
(ext_id, entry, loc);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> ACE_Unbounded_Set<INT_ID>
int_id_set;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> if (result == -1)<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> {<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> void *ptr;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> // Not found.<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> ACE_ALLOCATOR_RETURN (ptr,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
this->entry_allocator_->malloc (sizeof (ACE_Hash_Multi_Map_Entry<EXT_ID,
INT_ID>)),<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> -1);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> int_id_set.insert (int_id);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> entry = new (ptr)
ACE_Hash_Multi_Map_Entry<EXT_ID, INT_ID> (ext_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
int_id_set, </span><span lang=EN-US style='font-family:Wingdings'>ß</span><span
lang=EN-US> Unnecessary copying<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
this->table_[loc].next_,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
&this->table_[loc]);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> this->table_[loc].next_ = entry;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> entry->next_->prev_ = entry;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> this->cur_size_++;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> return 0;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> }<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> else<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> {<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> // Just unbelieveable way of new
element inserting<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> int_id_set = (*entry).int_id_set_;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> if (0 == int_id_set.insert (int_id))<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> {<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> this->unbind_i (entry);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> return this->bind_i (ext_id,
int_id_set);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> </span>}<o:p></o:p></p>
<p class=MsoNormal> else<o:p></o:p></p>
<p class=MsoNormal> return 1;<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal>}<o:p></o:p></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> REPEAT BY:<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>[What you did to get the error; include
test program or session<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>transcript if at all possible. ]<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> SAMPLE FIX/WORKAROUND:<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>This is a straightforward fix, but more
correct implementation requires introducing an additional constructor for
ACE_Hash_Multi_Map_Entry without<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>const ACE_Unbounded_Set<INT_ID>
&int_id_set parameter and using it during an entry creation.<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US>template <class EXT_ID, class INT_ID,
class HASH_KEY, class COMPARE_KEYS, class ACE_LOCK> int<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>ACE_Hash_Multi_Map_Manager<EXT_ID,
INT_ID, HASH_KEY, COMPARE_KEYS, ACE_LOCK>::bind_i (const EXT_ID &ext_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> const
INT_ID &int_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
ACE_Hash_Multi_Map_Entry<EXT_ID, INT_ID> *&entry)<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>{<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> size_t loc;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> int result = this->shared_find
(ext_id, entry, loc);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> if (result == -1)<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> {<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> void *ptr;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> // Not found.<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> ACE_ALLOCATOR_RETURN (ptr,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
this->entry_allocator_->malloc (sizeof (ACE_Hash_Multi_Map_Entry<EXT_ID,
INT_ID>)),<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> -1);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> entry = new (ptr)
ACE_Hash_Multi_Map_Entry<EXT_ID, INT_ID> (ext_id,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
ACE_Unbounded_Set<INT_ID>(), <o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
this->table_[loc].next_,<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>
&this->table_[loc]);<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> this->table_[loc].next_ = entry;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> entry->next_->prev_ = entry;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> }<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US> if (!(*entry).int_id_set_.insert(int_id))
{<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> this->cur_size_++;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> return 0;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> } else<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US> return 1;<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US>}<o:p></o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US><o:p> </o:p></span></p>
<p class=MsoNormal><span lang=EN-US>/Boris<o:p></o:p></span></p>
</div>
</body>
</html>