[omniORB] (bug) problem with virtual inheritance in IDL-generated classes
Jason Nye
jnye@nbnet.nb.ca
Mon Aug 5 17:57:00 2002
--------------Boundary-00=_BNFC3X2QR4LH4006K1ZL
Content-Type: text/plain;
charset="iso-8859-15"
Content-Transfer-Encoding: 8bit
Hello all,
I've just found what looks like is a bug in how omni generates servant
classes from inherited IDL interfaces.
My System:
Platform: Linux 2.4.8
gcc version 3.0.4
omniORB 3.0.5
The problem arises when an exception is thrown from the constructor of a
servant that implements an IDL interface that is derived from another IDL
interface. The small example below shows what I mean. See the attached files
for a fully working test harness that exhibits the problem more clearly.
--- IDL FILE ----
interface A {
...
};
interface B : A {
...
};
--- END IDL FILE ---
--- C++ Servant & main program ----
...
using namespace std;
class Servant_impl : public virtual POA_B
{
public:
Servant_impl() { throw runtime_error("dummy error"); }
....
};
int main(int argc, char * argv[])
try {
CORBA::ORB_var orb = ...
Servant_impl s;
....
orb->run();
return 0;
}
catch (const exception & ex) {
cout << "Error: " << ex.what() << endl;
return -1;
}
--- END C++ Servant & main program ---
The above program will cause segment violation. The interesting thing is that
if Servant_impl was derived from POA_A instead of POA_B, no problem would
result -- the exception would be thrown and caught as expected.
The attached test harness shows that this problem is not a compiler problem
as far as I can tell because I created interface-equivalent base classes for
the servant to derive from and when compiled to use those, no crash occurs.
Please try the test harness to see how it works (much documentation is
included in it). You may have to change some paths in the makefile.
Note that I did not run it through the debugger as I am not very good at all
at using gdb.
Thanks for your help,
Jason.
-------------------------------------------------------
--------------Boundary-00=_BNFC3X2QR4LH4006K1ZL
Content-Type: text/x-c++;
charset="iso-8859-15";
name="omni_bug_test.idl"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="omni_bug_test.idl"
I2lmbmRlZiBPTU5JX0JVR19URVNUX0lETAojZGVmaW5lIE9NTklfQlVHX1RFU1RfSURMCgoKLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8KLy8gQSBz
aW1wbGUgSURMIGZpbGUgY29udGFpbmluZyB0d28gaW50ZXJmYWNlczoKLy8gYnVnX3Rlc3RfYmFz
ZSBhbmQgYnVnX3Rlc3RfZGVyaXZlZCB3aGljaCBpcwovLyBkZXJpdmVkIGZyb20gYnVnX3Rlc3Rf
YmFzZS4gVGhlc2UgdHdvIGludGVyZmFjZXMKLy8gYXJlIHVzZWQgaW4gdGhlIHRlc3QgaGFybmVz
cyB0byBzaG93IHRoYXQKLy8gdGhlcmUgc2VlbXMgdG8gYmUgc29tZXRoaW5nIHdyb25nIHdpdGgg
T01OSSBPUkIncwovLyBkZXJpdmF0aW9uIG9mIGludGVyZmFjZXMuCi8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vCgoKaW50ZXJmYWNlIGJ1Z190ZXN0
X2Jhc2UgewoKICAgIHN0cmluZyBuYW1lKCk7Cgp9OwoKaW50ZXJmYWNlIGJ1Z190ZXN0X2Rlcml2
ZWQgOiBidWdfdGVzdF9iYXNlIHsKCiAgICBzdHJpbmcgbmFtZTIoKTsKCn07CgojZW5kaWYKCg==
--------------Boundary-00=_BNFC3X2QR4LH4006K1ZL
Content-Type: text/x-c++;
charset="iso-8859-15";
name="omni_bug_test.cpp"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="omni_bug_test.cpp"
Ly8KLy8gVGVzdCBoYXJuZXNzIHRvIHNob3cgdGhhdCB0aGVyZSBpcyBhIHByb2JsZW0gd2l0aCB2
aXJ0dWFsCi8vIGluaGVyaXRhbmNlIGluIE9NTkkgT1JCIDMuMC41IElETC1nZW5lcmF0ZWQgY2xh
c3NlcyAoYW4KLy8gZXhjZXB0aW9uIHRocm93biBmcm9tIHRoZSBjb25zdHJ1Y3RvciBvZiBhbiBp
bXBsZW1lbnRlZAovLyBzZXJ2YW50IHdpbGwgY3Jhc2ggdGhlIHByb2dyYW0gaWYgdGhlIHNlcnZh
bnQgaW1wbGVtZW50cwovLyBhbiBJREwgaW50ZXJmYWNlIHdoaWNoIGlzLCBkZXJpdmVkIGZyb20g
YW5vdGhlciBJREwgaW50ZXJmYWNlKS4KLy8KLy8gVGhlIHRlc3QgaGFybmVzcyB1c2VzIGRlZmlu
ZXMgdGhhdCBhbGxvdyB5b3UgdG8gY2hvb3NlIGJhc2UKLy8gc2VydmFudCBjbGFzc2VzIHRoYXQg
YXJlIElETC1nZW5lcmF0ZWQgb3IgaW50ZXJmYWNlLWVxdWl2YWxlbnRzCi8vIHRoYXQgYXJlIGRl
ZmluZWQgaW4gdGhpcyBmaWxlLiBCeSBwbGF5aW5nIGFyb3VuZCB3aXRoIHdpdGggdGhlCi8vIGRl
ZmluZXMgYXMgc2hvd24gYmVsb3csIHlvdSBtYXkgYmUgYWJsZSB0byByZXByb2R1Y2UgdGhlIGJl
aGF2aW91cgovLyB0aGF0IEkgb2JzZXJ2ZWQgKGp1c3QgY2hhbmdlIHRoZSBkZWZpbmVzIGFzIGRl
c2NyaWJlZCBiZWxvdywKLy8gcmVjb21waWxlIGFuZCBydW4gdGhlIHRlc3QpLgovLwojaW5jbHVk
ZSA8c3RkZXhjZXB0PgojaW5jbHVkZSA8aW9zdHJlYW0+CgoKLyoKCkJlbG93IGFyZSB0d28gZGVm
aW5lcyB0aGF0IGFsbG93IHlvdSB0byBjb250cm9sIHdoaWNoCmNsYXNzZXMgYXJlIHVzZWQgYW5k
IGFsbG93IHlvdSB0byB0ZXN0IHdoaWNoIGNvbWJpbmF0aW9ucwpjYXVzZSB0aGUgcHJvZ3JhbSB0
byBjcmFzaC4gVGhlIGZvbGxvd2luZyB0YWJsZSBzaG93cwp3aGF0IEkgZXhwZXJpZW5jZWQ6CgpV
U0VfSURMX0NMQVNTRVMgICAgfCBVU0VfQkFTRV9JTlNURUFEX09GX0RFUklWRUQgICB8IEJFSEFW
SU9VUgo9PT09PT09PT09PT09PT09PT09Kz09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0r
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KICAgICAgMSAgICAgICAgICAg
IHwgICAgICAgICAgICAgIDAgICAgICAgICAgICAgICAgfCBTRUdNRU5UIFZJT0xBVElPTgotLS0t
LS0tLS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0rLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KICAgICAgMSAgICAgICAgICAgIHwgICAgICAg
ICAgICAgIDEgICAgICAgICAgICAgICAgfCBFWFBFQ1RFRCBCRUhBVklPVVIgKEVYQ0VQVElPTiBU
SFJPV04pCi0tLS0tLS0tLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LSstLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQogICAgICAwICAgICAgICAg
ICAgfCAgICAgICAgICAgICAgMCAgICAgICAgICAgICAgICB8IEVYUEVDVEVEIEJFSEFWSU9VUiAo
RVhDRVBUSU9OIFRIUk9XTikKLS0tLS0tLS0tLS0tLS0tLS0tLSstLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tKy0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tCiAgICAg
IDAgICAgICAgICAgICB8ICAgICAgICAgICAgICAxICAgICAgICAgICAgICAgIHwgRVhQRUNURUQg
QkVIQVZJT1VSIChFWENFUFRJT04gVEhST1dOKQotLS0tLS0tLS0tLS0tLS0tLS0tKy0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0rLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0KICovCgovLyBTZXQgdGhlIGZvbGxvd2luZyB0byAxIGlmIHlvdSB3YW50IHRvIGNvbXBp
bGUgd2l0aAovLyBpZGwtZ2VuZXJhdGVkIGNsYXNzZXMsIDAgaWYgeW91IHdhbnQgdG8gY29tcGls
ZSB3aXRoCi8vIHRoZSB0ZXN0IGNsYXNzZXMgaW4gdGhpcyBmaWxlIChlcXVpdmFsZW50cyB3aXRo
IHRoZQovLyBpZGwtZ2VuZXJhdGVkIGNsYXNzZXMgYXMgZmFyIGFzIHRoZSBpbnRlcmZhY2UgZ29l
cykuCiNkZWZpbmUgVVNFX0lETF9DTEFTU0VTIDEKCgovLyBTZXQgdGhlIGZvbGxvd2luZyB0byAx
IGlmIHlvdSB3YW50IHRvIHVzZSB0aGUgYmFzZQovLyBjbGFzcyAoZWl0aGVyIFBPQV9idWdfdGVz
dF9iYXNlIGlmIFVTRV9JRExfQ0xBU1NFUz09MSwKLy8gb3IgYmFzZSwgYmVsb3csIGlmIFVTRV9J
RExfQ0xBU1NFUz09MCkuCiNkZWZpbmUgVVNFX0JBU0VfSU5TVEVBRF9PRl9ERVJJVkVEIDAKCgp1
c2luZyBuYW1lc3BhY2Ugc3RkOwoKCiNpZiBVU0VfSURMX0NMQVNTRVM9PTEKICAgICNpbmNsdWRl
ICJvbW5pX2J1Z190ZXN0LmhoIgogICAgI2lmIFVTRV9CQVNFX0lOU1RFQURfT0ZfREVSSVZFRD09
MQogICAgICAgICNkZWZpbmUgIEJBU0VfQ0xBU1MgUE9BX2J1Z190ZXN0X2Jhc2UKICAgICNlbHNl
CiAgICAgICAgI2RlZmluZSAgQkFTRV9DTEFTUyBQT0FfYnVnX3Rlc3RfZGVyaXZlZAogICAgI2Vu
ZGlmCiNlbHNlCiAgICAvLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLy8vLy8vLy8vLy8vLy8vCiAgICAvLyBUaGUgZm9sbG93aW5nIHR3byBjbGFzc2VzIGJhc2Ug
JiBkZXJpdmVkIGFyZQogICAgLy8gZXF1aXZhbGVudHMgdG8gUE9BX2J1Z190ZXN0X2Jhc2UgYW5k
IFBPQV9idWdfdGVzdF9kZXJpdmVkCiAgICAvLyBnZW5lcmF0ZWQgZnJvbSB0aGUgSURMLgogICAg
Ly8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8vLy8v
Ly8vLwogICAgY2xhc3MgYmFzZSB7CiAgICBwdWJsaWM6CiAgICAgICAgYmFzZSgpIDogdmFsXyhu
ZXcgaW50KSB7fQogICAgICAgIHZpcnR1YWwgfmJhc2UoKSB7IGRlbGV0ZSB2YWxfOyB9CiAgICAg
ICAgdmlydHVhbCBjaGFyICogbmFtZSgpID0gMDsKICAgIHByaXZhdGU6CiAgICAgICAgLy8gSnVz
dCB3YW50ZWQgdGhlIGJhc2UgY2xhc3MgdG8gaGF2ZSBhIHZhbHVlIG9mIHNvbWUgc29ydC4KICAg
ICAgICBpbnQgKiB2YWxfOwogICAgfTsKCiAgICBjbGFzcyBkZXJpdmVkIDogcHVibGljIHZpcnR1
YWwgYmFzZSB7CiAgICBwdWJsaWM6CiAgICAgICAgZGVyaXZlZCgpIHt9CiAgICAgICAgdmlydHVh
bCBjaGFyICogbmFtZTIoKSA9IDA7CiAgICB9OwoKICAgICNpZiBVU0VfQkFTRV9JTlNURUFEX09G
X0RFUklWRUQ9PTEKICAgICAgICAjZGVmaW5lICBCQVNFX0NMQVNTIGJhc2UKICAgICNlbHNlCiAg
ICAgICAgI2RlZmluZSAgQkFTRV9DTEFTUyBkZXJpdmVkCiAgICAjZW5kaWYKI2VuZGlmCgoKCmNs
YXNzIGJ1Z190ZXN0X2ltcGwgOiBwdWJsaWMgQkFTRV9DTEFTUyB7CnB1YmxpYzoKICAgIGJ1Z190
ZXN0X2ltcGwoKQogICAgewogICAgICAgIC8vIFRoaXMgY2F1c2VzIGEgc2VnbWVudGF0aW9uIHZp
b2xhdGlvbiBpZiBVU0VfSURMX0NMQVNTRVM9PTEKICAgICAgICAvLyBhbmQgVVNFX0JBU0VfSU5T
VEVBRF9PRl9ERVJJVkVEPT0wCiAgICAgICAgdGhyb3cgcnVudGltZV9lcnJvcigiYnVnIHRlc3Qg
ZXJyb3IiKTsKICAgIH0KCiAgICB2aXJ0dWFsIGNoYXIgKiBuYW1lKCkKI2lmIFVTRV9JRExfQ0xB
U1NFUz09MQogICAgeyByZXR1cm4gQ09SQkE6OnN0cmluZ19kdXAoImJ1Z190ZXN0Iik7IH0KI2Vs
c2UKICAgIHsgcmV0dXJuICJidWdfdGVzdCI7IH0KI2VuZGlmCgogICAgdmlydHVhbCBjaGFyICog
bmFtZTIoKQojaWYgVVNFX0lETF9DTEFTU0VTPT0xCiAgICB7IHJldHVybiBDT1JCQTo6c3RyaW5n
X2R1cCgiYnVnX3Rlc3QyIik7IH0KI2Vsc2UKICAgIHsgcmV0dXJuICJidWdfdGVzdDIiOyB9CiNl
bmRpZgp9OwoKCmludCBtYWluKGludCBhcmdjLCBjaGFyICogYXJndltdKQp0cnkgewogICAgYnVn
X3Rlc3RfaW1wbCBpbXBsOwogICAgcmV0dXJuIDA7Cn0KY2F0Y2ggKGNvbnN0IGV4Y2VwdGlvbiAm
IGVycm9yKSB7CiAgICBjb3V0IDw8ICJlcnJvcjogIiA8PCBlcnJvci53aGF0KCkgPDwgZW5kbDsK
ICAgIHJldHVybiAwOwp9Cgo=
--------------Boundary-00=_BNFC3X2QR4LH4006K1ZL
Content-Type: text/x-makefile;
charset="iso-8859-15";
name="Makefile"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="Makefile"
IyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjCiMgTWFrZWZpbGUgZm9yIExpbnV4CiMj
IyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIwoKZXhwb3J0IE9QVElPTlMgPSAtZm5vLXJ0
dGkgLWZuby1lbGlkZS1jb25zdHJ1Y3RvcnMgLU8wIC1ETkRFQlVHCiNleHBvcnQgT1BUSU9OUyA9
IC1nCmV4cG9ydCBXQVJOSU5HUyA9IC1XYWxsCmV4cG9ydCBDRkxBR1MgPSAkKFdBUk5JTkdTKSAk
KE9QVElPTlMpIC1JLiBcCiAgICAgICAgICAgICAgICAtRF9SRUVOVFJBTlQgLURfR05VX1NPVVJD
RSAtZnBpYyAtcGlwZSAtRF9feDg2X18gXAogICAgICAgICAgICAgICAgLURfX09TVkVSU0lPTl9f
PTIgXAoKZXhwb3J0IENYWCA9IGcrKwpleHBvcnQgU0hFTEwgPSAvYmluL3NoCmV4cG9ydCBJREwg
PSBvbW5paWRsCkxERkxBR1MgPSAtTC9vbW5pL2xpYi9pNTg2X2xpbnV4XzIuMF9nbGliYzIuMSAt
bG9tbmlPUkIzIC1sQ09TMwoKLlNVRkZJWEVTOiAuY3BwCgpJRExTID0gICAgICBvbW5pX2J1Z190
ZXN0LmlkbApIRUFERVJTID0gICBvbW5pX2J1Z190ZXN0U0suaGgKU09VUkNFUyA9ICAgb21uaV9i
dWdfdGVzdFNLLmNjIG9tbmlfYnVnX3Rlc3QuY3BwCk9CSkVDVFMgPSAgIG9tbmlfYnVnX3Rlc3RT
Sy5vIG9tbmlfYnVnX3Rlc3QubwpUQVJHRVQgPSAgICBvbW5pX2J1Z190ZXN0CgouY2MubzoKCSQo
Q1hYKSAtYyAkKENGTEFHUykgJDwKCi5jcHAubzoKCSQoQ1hYKSAtYyAkKENGTEFHUykgJDwKCiMj
IyMjIyMgQnVpbGQgcnVsZXMKCi5QSE9OWTogY2xlYW4KLlBIT05ZOiBkb2NzCgokKFRBUkdFVCk6
ICQoU09VUkNFUykgJChPQkpFQ1RTKQoJJChDWFgpICQoT0JKRUNUUykgLW8gJChUQVJHRVQpICQo
TERGTEFHUykKCm9tbmlfYnVnX3Rlc3RTSy5jYzogb21uaV9idWdfdGVzdC5pZGwKCSQoSURMKSAt
YmN4eCAkPAoKY2xlYW46CglybSAtZiAqLm8gKi5iYWsgKn4gKiUKCXJtIC1mICQoVEFSR0VUKQoJ
cm0gLXJmIGRvYy8qCg==
--------------Boundary-00=_BNFC3X2QR4LH4006K1ZL--