390764 - xpidl crashes creating a typelib when struct or sequence is used in a idl file [@ SizeOfTypeDescriptor]

Status: RESOLVED INCOMPLETE

(Core :: XPCOM, defect)

Description

[koen]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1.5) Gecko/20070719 Iceweasel/2.0.0.5 (Debian-2.0.0.5-0etch1)
Build Identifier: Mozilla XULRunner 1.9a7pre - 2007073112

In the example below, if the attributes a and b remain commented all is well.
However if one or both are uncommented, xpidl gives a segmentation fault.
I also tested with xpidl 1.8 with the same result.


#include "nsISupports.idl"

struct aStruct {
  double x;
  double y;
};
  
typedef sequence<double> aseq;

[scriptable, uuid(7cb5b7a1-07d7-11d3-bde2-000064657376)]
interface nsITest : nsISupports {
  //attribute aStruct a;
  //attribute aseq b;
  
  attribute string c;
};


Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Andrew Schultz]

worksforme with current CVS on linux/x86 (32bit)

I get:
./foo.idl:8: Warning: sequences not supported, ignored

for the typedef line, but no crash even with both attribute lines uncommented.

Comment 2

[koen]

Sorry, I was somewhat unclear.
When creating a header file I get the same result as you.
However when running with '-m typelib' it crashes. (see below)

Do you know whether sequences will be implemented in the future?

root@localhost:~/projects/cmgui/trunk/xpcom/src# /root/projects/mozilla/obj-x86_64-unknown-linux-gnu/dist/gecko-sdk/bin/xpidl -w -m header -I/root/projects/mozilla/obj-x86_64-unknown-linux-gnu/dist/gecko-sdk/idl nsINode.idl
./nsINode.idl:8: Warning: sequences not supported, ignored

root@localhost:~/projects/cmgui/trunk/xpcom/src# /root/projects/mozilla/obj-x86_64-unknown-linux-gnu/dist/gecko-sdk/bin/xpidl -w -m typelib -I/root/projects/mozilla/obj-x86_64-unknown-linux-gnu/dist/gecko-sdk/idl nsINode.idl
Segmentation fault

Comment 3

[Andrew Schultz]

Attachment: stack

521         switch (XPT_TDP_TAG(td->prefix)) {
(gdb) p td
$1 = (XPTTypeDescriptor *) 0x2
(gdb) frame 1
553         size += 1 + SizeOfTypeDescriptor(&md->result->type, id);
(gdb) p md
$2 = (XPTMethodDescriptor *) 0x8fb8430
(gdb) p md->result
$3 = (XPTParamDescriptor *) 0x0

I get the same stack for the struct and the sequence.

Comment 4

[Nathan Froyd [:froydnj]]

C-based xpidl is long gone.