Minor bug: Segmentation fault on getinfo for a unfilled value
This commit is contained in:
parent
e089e15d85
commit
441b622709
2 changed files with 15 additions and 6 deletions
|
@ -1,23 +1,32 @@
|
|||
Curl = require('../index');
|
||||
options = {CONNECTTIMEOUT: 2};
|
||||
keys = ["EFFECTIVE_URL", "CONTENT_TYPE", "PRIVATE", "FTP_ENTRY_PATH", "REDIRECT_URL", "PRIMARY_IP", "RTSP_SESSION_ID", "LOCAL_IP"]
|
||||
curl = Curl.create(options)
|
||||
url = 'www.nodejs.org';
|
||||
curl(url, function(err) {
|
||||
console.info("\x1b[33meffetcive url: " + this.info('EFFECTIVE_URL') + "\x1b[0m");
|
||||
self = this
|
||||
keys.forEach(function(key) {
|
||||
console.info("\x1b[33m" + key + ": [" + self.info(key) + "]\x1b[0m");
|
||||
})
|
||||
console.info("body length: " + this.body.length);
|
||||
this.close()
|
||||
});
|
||||
|
||||
curl = Curl.create(options)
|
||||
url = 'www.yahoo.com'
|
||||
curl(url, function(err) {
|
||||
console.info("\x1b[33meffetcive url: " + this.info('EFFECTIVE_URL') + "\x1b[0m");
|
||||
self = this
|
||||
keys.forEach(function(key) {
|
||||
console.info("\x1b[33m" + key + ": [" + self.info(key) + "]\x1b[0m");
|
||||
})
|
||||
console.info("body length: " + this.body.length);
|
||||
this.close()
|
||||
});
|
||||
|
||||
curl = Curl.create(options)
|
||||
curl('https://www.google.com', {VERBOSE: 1, RAW: 1}, function(err) {
|
||||
console.info("\x1b[33meffetcive url: " + this.info('EFFECTIVE_URL') + "\x1b[0m");
|
||||
self = this
|
||||
keys.forEach(function(key) {
|
||||
console.info("\x1b[33m" + key + ": [" + self.info(key) + "]\x1b[0m");
|
||||
})
|
||||
console.info("body length: " + this.body.length);
|
||||
this.close()
|
||||
});
|
||||
|
|
|
@ -155,7 +155,7 @@ class NodeCurl
|
|||
{
|
||||
return raise("curl_easy_getinfo failed", curl_easy_strerror(code));
|
||||
}
|
||||
return JsClass::New(result);
|
||||
return result ? JsClass::New(result) : v8::Null();
|
||||
}
|
||||
|
||||
static v8::Handle<v8::Value> getinfo_int(const v8::Arguments & args)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue