Export to GitHub

v8cgi - issue #80

Access violation


Posted on Jan 4, 2011 by Helpful Wombat

What steps will reproduce the problem? 1. system.stdout(new Date) 2. system.stdout({}) 3.

What is the expected output? What do you see instead? the printed message but i see a system message alert: access violation

What version of the product are you using? On what operating system? v8cgi 0.9 precompiled for windows on my windows xp it

Please provide any additional information below. when i pass an object to the method instead of a string

Comment #1

Posted on Jan 4, 2011 by Happy Giraffe

I am unable to reproduce this - can you please provide a screenshot + any additional details available for the access violation alert?

Comment #2

Posted on Jan 4, 2011 by Helpful Wombat

v8cgi -c v8cgi.conf a.js

where a.js is a single line: system.stdout(new Date)

and v8cgi.conf is: var Config = {}; exports.Config = Config;

// put default .js and .so files here // NOTE: this was called "Config.libraryPath" in older versions of v8cgi require.paths.push("c:/program files/v8cgi/lib");

// these will get loaded automatically Config["libraryAutoload"] = ["js", "html", "http"];

// name of session cookie Config["sessionCookie"] = "V8SID";

// directory for session storage Config["sessionPath"] = "c:/windows/temp";

// session lifetime - in seconds Config["sessionTime"] = 60*60;

// session domain Config["sessionDomain"] = "/";

// address for sending emails Config["smtpHost"] = "127.0.0.1";

// port for sending emails Config["smtpPort"] = 25;

// default From: header Config["smtpFrom"] = "";

// Uncaught exceptions go to stdout (true) or stderr (false)

Config["showErrors"] = true;

system.env contains:

F:\V8Cgi>v8cgi -c v8cgi.conf a.js ALLUSERSPROFILE: C:\Documents and Settings\All Users APPDATA: C:\Documents and Settings\TEMP\Dati applicazioni CLIENTNAME: Console CommonProgramFiles: C:\Programmi\File comuni COMPUTERNAME: * ComSpec: C:\WINDOWS\system32\cmd.exe FP_NO_HOST_CHECK: NO FTP_PASSIVE: 1 HOMEDRIVE: C: HOMEPATH: \Documents and Settings\TEMP LOGONSERVER: * NUMBER_OF_PROCESSORS: 2 OPENSSL_CONF: C:\OpenSSL-Win32\bin\openssl.cfg OS: Windows_NT Path: * perl\bin;C:\Programmi\EasyFrom Trial PATHEXT: .COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.PSC1;.PSC1 PROCESSOR_ARCHITECTURE: x86 PROCESSOR_IDENTIFIER: x86 Family 15 Model 6 Stepping 5, GenuineIntel PROCESSOR_LEVEL: 15 PROCESSOR_REVISION: 0605 ProgramFiles: C:\Programmi PROMPT: $P$G PSModulePath: C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules\ SESSIONNAME: Console SystemDrive: C: SystemRoot: C:\WINDOWS TEMP: c:\ TERM: dumb TMP: c:\ USERDNSDOMAIN: * USERDOMAIN: * USERNAME: * USERPROFILE: C:\Documents and Settings\TEMP VS80COMNTOOLS: C:\Programmi\Microsoft Visual Studio 8\Common7\Tools\ VS90COMNTOOLS: C:\Programmi\Microsoft Visual Studio 9.0\Common7\Tools\ WEBKITLIBRARIESDIR: F:\Tools\WebKit\WebKitLibraries\win WEBKITOUTPUTDIR: F:\Tools\WebKit\Output windir: C:\WINDOWS

Attachments

Comment #3

Posted on Jan 4, 2011 by Happy Giraffe

Problem reproduced. This is most probably caused by the fact that you specify an invalid path in require.paths.push (config file).

However, the crash should be probably changed to some more user-friendly way of reporting a configuration error :)

Comment #4

Posted on Jan 4, 2011 by Helpful Wombat

I can confirm that setting require.paths.push to a valid path remove the crash Thanks :)

Comment #5

Posted on Jan 5, 2011 by Happy Giraffe

I commited a crash-fix in revision 892. If you wish, I can build you a new windows binary - but I presume that is not necessary now, when you corrected the require path. This fix will be of course present in future v8cgi releases.

Status: Fixed

Labels:
Type-Defect Priority-Medium