wqx081 / v8cgi

Automatically exported from code.google.com/p/v8cgi
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Access violation #80

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
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 

Original issue reported on code.google.com by nanoyo...@yahoo.it on 4 Jan 2011 at 7:50

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

Original comment by ondrej.zara on 4 Jan 2011 at 9:15

GoogleCodeExporter commented 8 years ago
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

Original comment by nanoyo...@yahoo.it on 4 Jan 2011 at 10:51

Attachments:

GoogleCodeExporter commented 8 years ago
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 :)

Original comment by ondrej.zara on 4 Jan 2011 at 2:54

GoogleCodeExporter commented 8 years ago
I can confirm that setting require.paths.push to a valid path remove the crash
Thanks :)

Original comment by nanoyo...@yahoo.it on 4 Jan 2011 at 3:57

GoogleCodeExporter commented 8 years ago
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.

Original comment by ondrej.zara on 5 Jan 2011 at 7:10