After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 321582 - Parameter Incompatabilities with xmlHashScanFull
Parameter Incompatabilities with xmlHashScanFull
Status: RESOLVED FIXED
Product: libxslt
Classification: Platform
Component: general
1.1.15
Other All
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2005-11-16 08:33 UTC by ralfjunker
Modified: 2005-11-16 11:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description ralfjunker 2005-11-16 08:33:07 UTC 
Steps to reproduce:
I am not using libxslt outside of gnome. The bug shows when the library is
compiled not using __cdecl but __fastcall instead, which passes parameters in
registers. This requires that the the parameter list of the called function is
compatible with that of the callee. I propse a fix below.

Stack trace:


Other information:
In libexslt, file functions.c, the functions

* exsltFuncRegisterFunc 
* exsltFuncRegisterImportFunc 

need an additional parameter to be compatible with function xmlHashScanFull.

I propose the following changes:

static void 
exsltFuncRegisterFunc (exsltFuncFunctionData *data,
		       xsltTransformContextPtr ctxt,
		       const xmlChar *URI, const xmlChar *name /* DI */,
		       ATTRIBUTE_UNUSED const xmlChar *ignored /* DI */) {


static void
exsltFuncRegisterImportFunc (exsltFuncFunctionData *data,
			     exsltFuncImportRegData *ch,
			     const xmlChar *URI, const xmlChar *name /* DI */,
		       ATTRIBUTE_UNUSED const xmlChar *ignored /* DI */) {
Comment 1 Daniel Veillard 2005-11-16 11:48:48 UTC 
Okay those are static functions, so making that change is fairly easy.
Criticallity back to normal, because it is not on a GNOME platform, i.e.
from a GNOME point of view this is not a blocker.

  Fix appliced and commited,

 thanks a lot,

Daniel