<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from rtf -->
<style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<font face="Calibri" size="2"><span style="font-size:11pt;">
<div>Hi Duncan,</div>
<div> </div>
<div>Thanks for your valuable time and the prompt response.</div>
<div> </div>
<div><b>Platfrom</b>: AIX6.1</div>
<div> </div>
<div><b>You need to figure out why your build broke the exception handling</b><b>:</b></div>
<div> </div>
<div>Dynamic exception specification <b>throw(type)</b> is deprecated in C++17<b> </b><a href="https://en.cppreference.com/w/cpp/language/except_spec"><font color="#0563C1"><u>https://en.cppreference.com/w/cpp/language/except_spec</u></font></a></div>
<div> </div>
<div>When compiled the OmniORB-4.2.2 source with “gcc8 and C++17”, we resolved the compilation errors related to dynamic exception specifications those are unsupported in C++17. Mentioned few headers below,</div>
<div><font color="#833C0B">virtual void visit(const char* value,Source source) <b>throw (BadParam)</b> = 0;</font></div>
<div><font color="#833C0B">void addOptions(const char* options[][2]) <b>throw (Unknown,BadParam)</b>;</font></div>
<div>Removed all the dynamic exception specifications (“<font color="#833C0B"><b>throw</b></font>”) to compile with C++17 below,</div>
<div><font color="#833C0B">virtual void visit(const char* value,Source source) = 0;</font></div>
<div><font color="#833C0B">void addOptions(const char* options[][2]);</font></div>
<div> </div>
<div>If we remove the exception specification <font color="#833C0B"><b>throw (BadParam)</b></font><font color="#833C0B"><b> </b></font>from function header, it does allow the function to throw exception at any type.</div>
<div>So we thought it should not impact anything. </div>
<div> </div>
<div>Does this change break the exception handling? If so, how to resolve this issue and make it work with C++17?</div>
<div> </div>
<div>Thanks & Regards,</div>
<div>P. Prasath.</div>
<a name="_MailEndCompose"></a>
<div> </div>
<div>-----Original Message-----<br>
From: Duncan Grisby <duncan@grisby.org> <br>
Sent: Friday, March 29, 2019 3:50 PM<br>
To: Prasath Palaniappan <Prasath_Palaniappan@amat.com>; omniorb-list@omniorb-support.com<br>
Cc: Shankar Chinnusamy <Shankar_Chinnusamy@amat.com><br>
Subject: [External] Re: [omniORB] OmniORB-4.2.2 compiled with gcc8 and C++14 leads into crash</div>
<div> </div>
<div> </div>
<div>CAUTION: EXTERNAL EMAIL. Verify before you click links or open attachments. Questions? Contact GIS.</div>
<div> </div>
<div> </div>
<div> </div>
<div>On Mon, 2019-03-25 at 06:19 +0000, Prasath via omniORB-list wrote:</div>
<div> </div>
<div>[...]</div>
<div>> But now for project need, we have upgraded the compiler and std </div>
<div>> library from “gcc4.8 and C++14” to “gcc8 and C++17”. So that we </div>
<div>> compiled the source OmniORB-4.2.2 with “gcc8 and C++17”.</div>
<div> </div>
<div>[...]</div>
<div>> We tested the project (Our Application + OmniORB-4.2.2) which is </div>
<div>> completely compiled with “gcc8 and C++17”. But the process got crashed </div>
<div>> with terminate caught due to throwing dynamic exception on </div>
<div>> communication failure.</div>
<div> </div>
<div>Something is clearly wrong with your build of it then.</div>
<div> </div>
<div>[...]</div>
<div>> omniORB: Orderly connection shutdown:</div>
<div>> giop:tcp:[::ffff:10.41.46.93]:33260</div>
<div>> omniORB: throw giopStream::CommFailure from</div>
<div>> giopImpl12.cc:1243(0,NO,COMM_FAILURE_UnMarshalArguments)</div>
<div> </div>
<div>This is omniORB's normal way of handling a connection being closed. It throws the internal giopStream::CommFailure exception and catches it higher up the call chain.</div>
<div> </div>
<div>> Terminate caught</div>
<div> </div>
<div>But this shows that the exception handler didn't trigger. The problem is not in omniORB, but something about the way it was compiled.</div>
<div> </div>
<div>[...]</div>
<div>> Turnoff the exception – We commented out the exception throw code at </div>
<div>> line number:512 in “src\lib\omniORB\orbcore\giopStream.cc”</div>
<div>> throw CommFailure(minor,status,retry);</div>
<div> </div>
<div>Clearly that is not the right thing to do. The code throws the exception on purpose. If you break the code by no longer throwing the exception, the server will no longer properly clean up connections.</div>
<div> </div>
<div>You need to figure out why your build broke the exception handling.</div>
<div> </div>
<div>You haven't mentioned what platform you are using.</div>
<div> </div>
<div>Duncan.</div>
<div> </div>
<div>--</div>
<div> -- Duncan Grisby --</div>
<div> -- <a href="mailto:duncan@grisby.org">duncan@grisby.org</a> --</div>
<div> -- <a href="http://www.grisby.org">http://www.grisby.org</a> --</div>
<div> </div>
<div> </div>
<div><font face="Calibri">The content of this message is APPLIED MATERIALS CONFIDENTIAL. If you are not the intended recipient, please notify me, delete this email and do not use or distribute this email. </font></div>
</span></font>
</body>
</html>