From 8e29ce9f13f7e4b6c0296325287c05f78ff22ce0 Mon Sep 17 00:00:00 2001 From: Fedor Indutny Date: Wed, 31 Jul 2013 22:07:29 +0400 Subject: src: lint c++ code --- src/cares_wrap.cc | 41 ++--- src/fs_event_wrap.cc | 18 +- src/handle_wrap.h | 16 +- src/node.cc | 115 ++++++------- src/node.h | 4 +- src/node_buffer.cc | 2 - src/node_buffer.h | 8 +- src/node_constants.cc | 332 +++++++++++++++++++------------------ src/node_constants.h | 6 +- src/node_counters.cc | 5 +- src/node_counters.h | 8 +- src/node_crypto.cc | 271 +++++++++++++++--------------- src/node_crypto.h | 25 ++- src/node_crypto_bio.cc | 88 +++++----- src/node_crypto_bio.h | 7 +- src/node_crypto_groups.h | 25 +++ src/node_dtrace.cc | 2 +- src/node_dtrace.h | 39 +++-- src/node_extensions.cc | 10 +- src/node_extensions.h | 57 ++++--- src/node_file.cc | 20 +-- src/node_file.h | 7 +- src/node_http_parser.cc | 19 +-- src/node_http_parser.h | 8 +- src/node_internals.h | 42 +++-- src/node_javascript.cc | 12 +- src/node_javascript.h | 5 + src/node_object_wrap.h | 14 +- src/node_os.cc | 28 ++-- src/node_os.h | 11 +- src/node_script.cc | 5 +- src/node_script.h | 10 +- src/node_stat_watcher.cc | 8 +- src/node_stat_watcher.h | 6 +- src/node_version.h | 9 +- src/node_watchdog.cc | 5 +- src/node_watchdog.h | 2 +- src/node_win32_etw_provider-inl.h | 28 ++-- src/node_win32_etw_provider.cc | 4 +- src/node_win32_etw_provider.h | 15 +- src/node_win32_perfctr_provider.cc | 36 ++-- src/node_win32_perfctr_provider.h | 9 +- src/node_wrap.h | 8 +- src/node_zlib.cc | 35 ++-- src/pipe_wrap.cc | 18 +- src/pipe_wrap.h | 6 +- src/process_wrap.cc | 8 +- src/queue.h | 6 +- src/req_wrap.h | 10 +- src/signal_wrap.cc | 2 +- src/smalloc.h | 7 +- src/stream_wrap.cc | 12 +- src/stream_wrap.h | 10 +- src/string_bytes.cc | 4 +- src/tcp_wrap.cc | 16 +- src/tcp_wrap.h | 8 +- src/timer_wrap.cc | 2 +- src/tls_wrap.cc | 228 ++++++++++++------------- src/tty_wrap.cc | 4 +- src/tty_wrap.h | 8 +- src/udp_wrap.cc | 6 +- src/udp_wrap.h | 31 +++- src/v8abbr.h | 28 +++- 63 files changed, 964 insertions(+), 875 deletions(-) (limited to 'src') diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc index 663be89629..756e0bc300 100644 --- a/src/cares_wrap.cc +++ b/src/cares_wrap.cc @@ -42,7 +42,6 @@ namespace node { - namespace cares_wrap { using v8::Array; @@ -230,7 +229,7 @@ static Local HostentToNames(struct hostent* host) { class QueryWrap { public: - QueryWrap(Local req_wrap_obj) { + explicit QueryWrap(Local req_wrap_obj) { HandleScope scope(node_isolate); persistent().Reset(node_isolate, req_wrap_obj); } @@ -333,7 +332,7 @@ class QueryWrap { class QueryAWrap: public QueryWrap { public: - QueryAWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryAWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -363,7 +362,7 @@ class QueryAWrap: public QueryWrap { class QueryAaaaWrap: public QueryWrap { public: - QueryAaaaWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryAaaaWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -398,7 +397,8 @@ class QueryAaaaWrap: public QueryWrap { class QueryCnameWrap: public QueryWrap { public: - QueryCnameWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryCnameWrap(Local req_wrap_obj) + : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -436,7 +436,7 @@ class QueryCnameWrap: public QueryWrap { class QueryMxWrap: public QueryWrap { public: - QueryMxWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryMxWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -478,7 +478,7 @@ class QueryMxWrap: public QueryWrap { class QueryNsWrap: public QueryWrap { public: - QueryNsWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryNsWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -506,7 +506,7 @@ class QueryNsWrap: public QueryWrap { class QueryTxtWrap: public QueryWrap { public: - QueryTxtWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryTxtWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -541,7 +541,7 @@ class QueryTxtWrap: public QueryWrap { class QuerySrvWrap: public QueryWrap { public: - QuerySrvWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QuerySrvWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -593,7 +593,8 @@ class QuerySrvWrap: public QueryWrap { class QueryNaptrWrap: public QueryWrap { public: - QueryNaptrWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit QueryNaptrWrap(Local req_wrap_obj) + : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -630,7 +631,6 @@ class QueryNaptrWrap: public QueryWrap { for (ares_naptr_reply* naptr_current = naptr_start; naptr_current; naptr_current = naptr_current->next) { - Local naptr_record = Object::New(); naptr_record->Set(flags_symbol, @@ -658,7 +658,8 @@ class QueryNaptrWrap: public QueryWrap { class GetHostByAddrWrap: public QueryWrap { public: - GetHostByAddrWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit GetHostByAddrWrap(Local req_wrap_obj) + : QueryWrap(req_wrap_obj) { } int Send(const char* name) { @@ -695,7 +696,8 @@ class GetHostByAddrWrap: public QueryWrap { class GetHostByNameWrap: public QueryWrap { public: - GetHostByNameWrap(Local req_wrap_obj) : QueryWrap(req_wrap_obj) { + explicit GetHostByNameWrap(Local req_wrap_obj) + : QueryWrap(req_wrap_obj) { } int Send(const char* name, int family) { @@ -768,7 +770,8 @@ static void QueryWithFamily(const FunctionCallbackInfo& args) { void AfterGetAddrInfo(uv_getaddrinfo_t* req, int status, struct addrinfo* res) { HandleScope scope(node_isolate); - GetAddrInfoReqWrap* req_wrap = (GetAddrInfoReqWrap*) req->data; + GetAddrInfoReqWrap* req_wrap = + reinterpret_cast(req->data); Local argv[] = { Integer::New(status, node_isolate), @@ -802,7 +805,8 @@ void AfterGetAddrInfo(uv_getaddrinfo_t* req, int status, struct addrinfo* res) { // Ignore random ai_family types. if (address->ai_family == AF_INET) { // Juggle pointers - addr = (char*) &((struct sockaddr_in*) address->ai_addr)->sin_addr; + addr = reinterpret_cast(&(reinterpret_cast( + address->ai_addr)->sin_addr)); int err = uv_inet_ntop(address->ai_family, addr, ip, @@ -828,7 +832,8 @@ void AfterGetAddrInfo(uv_getaddrinfo_t* req, int status, struct addrinfo* res) { // Ignore random ai_family types. if (address->ai_family == AF_INET6) { // Juggle pointers - addr = (char*) &((struct sockaddr_in6*) address->ai_addr)->sin6_addr; + addr = reinterpret_cast(&(reinterpret_cast( + address->ai_addr)->sin6_addr)); int err = uv_inet_ntop(address->ai_family, addr, ip, @@ -1075,9 +1080,7 @@ static void Initialize(Handle target) { oncomplete_sym = String::New("oncomplete"); } - -} // namespace cares_wrap - +} // namespace cares_wrap } // namespace node NODE_MODULE(node_cares_wrap, node::cares_wrap::Initialize) diff --git a/src/fs_event_wrap.cc b/src/fs_event_wrap.cc index d33308b4db..3843f5db50 100644 --- a/src/fs_event_wrap.cc +++ b/src/fs_event_wrap.cc @@ -41,14 +41,14 @@ static Cached onchange_sym; static Cached rename_sym; class FSEventWrap: public HandleWrap { -public: + public: static void Initialize(Handle target); static void New(const FunctionCallbackInfo& args); static void Start(const FunctionCallbackInfo& args); static void Close(const FunctionCallbackInfo& args); -private: - FSEventWrap(Handle object); + private: + explicit FSEventWrap(Handle object); virtual ~FSEventWrap(); static void OnEvent(uv_fs_event_t* handle, const char* filename, int events, @@ -147,14 +147,11 @@ void FSEventWrap::OnEvent(uv_fs_event_t* handle, const char* filename, // unreasonable, right? Still, we should revisit this before v1.0. if (status) { eventStr = String::Empty(node_isolate); - } - else if (events & UV_RENAME) { + } else if (events & UV_RENAME) { eventStr = rename_sym; - } - else if (events & UV_CHANGE) { + } else if (events & UV_CHANGE) { eventStr = change_sym; - } - else { + } else { assert(0 && "bad fs events flag"); abort(); } @@ -190,7 +187,6 @@ void FSEventWrap::Close(const FunctionCallbackInfo& args) { HandleWrap::Close(args); } - -} // namespace node +} // namespace node NODE_MODULE(node_fs_event_wrap, node::FSEventWrap::Initialize) diff --git a/src/handle_wrap.h b/src/handle_wrap.h index d8194b97b0..48db9e7f81 100644 --- a/src/handle_wrap.h +++ b/src/handle_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef HANDLE_WRAP_H_ -#define HANDLE_WRAP_H_ +#ifndef SRC_HANDLE_WRAP_H_ +#define SRC_HANDLE_WRAP_H_ #include "queue.h" @@ -53,16 +53,16 @@ namespace node { args.This()->GetAlignedPointerFromInternalField(0)); class HandleWrap { -public: + public: static void Initialize(v8::Handle target); static void Close(const v8::FunctionCallbackInfo& args); static void Ref(const v8::FunctionCallbackInfo& args); static void Unref(const v8::FunctionCallbackInfo& args); - inline uv_handle_t* GetHandle() { return handle__; }; + inline uv_handle_t* GetHandle() { return handle__; } -protected: - HandleWrap(v8::Handle object, uv_handle_t* handle); + protected: + explicit HandleWrap(v8::Handle object, uv_handle_t* handle); virtual ~HandleWrap(); inline v8::Local object() { @@ -73,7 +73,7 @@ protected: return object_; } -private: + private: friend void GetActiveHandles(const v8::FunctionCallbackInfo&); static void OnClose(uv_handle_t* handle); v8::Persistent object_; @@ -91,4 +91,4 @@ private: } // namespace node -#endif // HANDLE_WRAP_H_ +#endif // SRC_HANDLE_WRAP_H_ diff --git a/src/node.cc b/src/node.cc index c54489d89e..ac15247747 100644 --- a/src/node.cc +++ b/src/node.cc @@ -80,12 +80,12 @@ typedef int mode_t; #endif #include "node_script.h" -# ifdef __APPLE__ +#ifdef __APPLE__ # include # define environ (*_NSGetEnviron()) -# elif !defined(_MSC_VER) +#elif !defined(_MSC_VER) extern char **environ; -# endif +#endif namespace node { @@ -164,7 +164,7 @@ static char *eval_string = NULL; static int option_end_index = 0; static bool use_debug_agent = false; static bool debug_wait_connect = false; -static int debug_port=5858; +static int debug_port = 5858; static int max_stack_size = 0; bool using_domains = false; @@ -208,7 +208,7 @@ Isolate* node_isolate = NULL; class ArrayBufferAllocator : public ArrayBuffer::Allocator { -public: + public: // Impose an upper limit to avoid out of memory errors that bring down // the process. static const size_t kMaxLength = 0x3fffffff; @@ -216,7 +216,7 @@ public: virtual ~ArrayBufferAllocator() {} virtual void* Allocate(size_t length); virtual void Free(void* data); -private: + private: ArrayBufferAllocator() {} ArrayBufferAllocator(const ArrayBufferAllocator&); void operator=(const ArrayBufferAllocator&); @@ -260,7 +260,6 @@ static void IdleImmediateDummy(uv_idle_t* handle, int status) { static inline const char *errno_string(int errorno) { #define ERRNO_CASE(e) case e: return #e; switch (errorno) { - #ifdef EACCES ERRNO_CASE(EACCES); #endif @@ -585,7 +584,6 @@ static inline const char *errno_string(int errorno) { const char *signo_string(int signo) { #define SIGNO_CASE(e) case e: return #e; switch (signo) { - #ifdef SIGHUP SIGNO_CASE(SIGHUP); #endif @@ -823,7 +821,7 @@ Local UVException(int errorno, Local obj = e->ToObject(); - // TODO errno should probably go + // TODO(piscisaureus) errno should probably go obj->Set(errno_symbol, Integer::New(errorno, node_isolate)); obj->Set(code_symbol, estring); if (path) obj->Set(errpath_symbol, path_str); @@ -921,7 +919,7 @@ MakeDomainCallback(const Handle object, const Handle callback, int argc, Handle argv[]) { - // TODO Hook for long stack traces to be made here. + // TODO(trevnorris) Hook for long stack traces to be made here. // lazy load domain specific symbols if (enter_symbol.IsEmpty()) { @@ -1003,7 +1001,7 @@ MakeCallback(const Handle object, const Handle callback, int argc, Handle argv[]) { - // TODO Hook for long stack traces to be made here. + // TODO(trevnorris) Hook for long stack traces to be made here. Local process = PersistentToLocal(process_p); if (using_domains) @@ -1204,8 +1202,6 @@ void DisplayExceptionLine(Handle message) { int start = message->GetStartColumn(); int end = message->GetEndColumn(); - // fprintf(stderr, "---\nsourceline:%s\noffset:%d\nstart:%d\nend:%d\n---\n", sourceline_string, start, end); - fprintf(stderr, "%s\n", sourceline_string); // Print wavy underline (GetUnderline is deprecated). for (int i = 0; i < start; i++) { @@ -1252,7 +1248,7 @@ static void ReportException(Handle er, Handle message) { } -static void ReportException(TryCatch& try_catch) { +static void ReportException(const TryCatch& try_catch) { ReportException(try_catch.Exception(), try_catch.Message()); } @@ -1371,13 +1367,11 @@ static void Umask(const FunctionCallbackInfo& args) { if (args.Length() < 1 || args[0]->IsUndefined()) { old = umask(0); umask(static_cast(old)); - - } else if(!args[0]->IsInt32() && !args[0]->IsString()) { + } else if (!args[0]->IsInt32() && !args[0]->IsString()) { return ThrowTypeError("argument must be an integer or octal string."); - } else { int oct; - if(args[0]->IsInt32()) { + if (args[0]->IsInt32()) { oct = args[0]->Uint32Value(); } else { oct = 0; @@ -1664,7 +1658,7 @@ static void InitGroups(const FunctionCallbackInfo& args) { } } -#endif // __POSIX__ && !defined(__ANDROID__) +#endif // __POSIX__ && !defined(__ANDROID__) void Exit(const FunctionCallbackInfo& args) { @@ -1773,8 +1767,8 @@ void DLOpen(const FunctionCallbackInfo& args) { return ThrowError("process.dlopen takes exactly 2 arguments."); } - Local module = args[0]->ToObject(); // Cast - String::Utf8Value filename(args[1]); // Cast + Local module = args[0]->ToObject(); // Cast + String::Utf8Value filename(args[1]); // Cast if (exports_symbol.IsEmpty()) { exports_symbol = String::New("exports"); @@ -1786,7 +1780,7 @@ void DLOpen(const FunctionCallbackInfo& args) { #ifdef _WIN32 // Windows needs to add the filename into the error message errmsg = String::Concat(errmsg, args[1]->ToString()); -#endif +#endif // _WIN32 ThrowException(Exception::Error(errmsg)); return; } @@ -1800,7 +1794,7 @@ void DLOpen(const FunctionCallbackInfo& args) { if (pos != NULL) { base = pos + 1; } -#else // Windows +#else // Windows for (;;) { pos = strpbrk(base, "\\/:"); if (pos == NULL) { @@ -1808,7 +1802,7 @@ void DLOpen(const FunctionCallbackInfo& args) { } base = pos + 1; } -#endif +#endif // __POSIX__ /* Strip the .node extension. */ pos = strrchr(base, '.'); @@ -1913,9 +1907,9 @@ void FatalException(Handle error, Handle message) { } -void FatalException(TryCatch& try_catch) { +void FatalException(const TryCatch& try_catch) { HandleScope scope(node_isolate); - // TODO do not call FatalException if try_catch is verbose + // TODO(bajtos) do not call FatalException if try_catch is verbose // (requires V8 API to expose getter for try_catch.is_verbose_) FatalException(try_catch.Exception(), try_catch.Message()); } @@ -1946,7 +1940,7 @@ static void Binding(const FunctionCallbackInfo& args) { // Append a string to process.moduleLoadList char buf[1024]; - snprintf(buf, 1024, "Binding %s", *module_v); + snprintf(buf, sizeof(buf), "Binding %s", *module_v); Local modules = PersistentToLocal(module_load_list); uint32_t l = modules->Length(); @@ -1958,19 +1952,15 @@ static void Binding(const FunctionCallbackInfo& args) { // only exports. modp->register_func(exports, Undefined(node_isolate)); cache->Set(module, exports); - } else if (!strcmp(*module_v, "constants")) { exports = Object::New(); DefineConstants(exports); cache->Set(module, exports); - } else if (!strcmp(*module_v, "natives")) { exports = Object::New(); DefineJavaScript(exports); cache->Set(module, exports); - } else { - return ThrowError("No such module"); } @@ -1992,7 +1982,7 @@ static void ProcessTitleSetter(Local property, const PropertyCallbackInfo& info) { HandleScope scope(node_isolate); String::Utf8Value title(value); - // TODO: protect with a lock + // TODO(piscisaureus): protect with a lock uv_set_process_title(*title); } @@ -2008,7 +1998,7 @@ static void EnvGetter(Local property, } #else // _WIN32 String::Value key(property); - WCHAR buffer[32767]; // The maximum size allowed for environment variables. + WCHAR buffer[32767]; // The maximum size allowed for environment variables. DWORD result = GetEnvironmentVariableW(reinterpret_cast(*key), buffer, ARRAY_SIZE(buffer)); @@ -2142,16 +2132,17 @@ static Handle GetFeatures() { HandleScope scope(node_isolate); Local obj = Object::New(); - obj->Set(String::NewSymbol("debug"), #if defined(DEBUG) && DEBUG - True(node_isolate) + Local debug = True(node_isolate); #else - False(node_isolate) -#endif - ); + Local debug = False(node_isolate); +#endif // defined(DEBUG) && DEBUG + + obj->Set(String::NewSymbol("debug"), debug); obj->Set(String::NewSymbol("uv"), True(node_isolate)); - obj->Set(String::NewSymbol("ipv6"), True(node_isolate)); // TODO ping libuv + // TODO(bnoordhuis) ping libuv + obj->Set(String::NewSymbol("ipv6"), True(node_isolate)); obj->Set(String::NewSymbol("tls_npn"), Boolean::New(use_npn)); obj->Set(String::NewSymbol("tls_sni"), Boolean::New(use_sni)); obj->Set(String::NewSymbol("tls"), @@ -2375,7 +2366,7 @@ Handle SetupProcessObject(int argc, char *argv[]) { NODE_SET_METHOD(process, "getgroups", GetGroups); NODE_SET_METHOD(process, "setgroups", SetGroups); NODE_SET_METHOD(process, "initgroups", InitGroups); -#endif // __POSIX__ && !defined(__ANDROID__) +#endif // __POSIX__ && !defined(__ANDROID__) NODE_SET_METHOD(process, "_kill", Kill); @@ -2488,7 +2479,7 @@ static void ParseDebugOpt(const char* arg) { debug_port = atoi(p); } else { use_debug_agent = true; - if (!strcmp (arg, "--debug-brk")) { + if (!strcmp(arg, "--debug-brk")) { debug_wait_connect = true; return; } else if (!strcmp(arg, "--debug")) { @@ -2546,7 +2537,7 @@ static void PrintHelp() { static void ParseArgs(int argc, char **argv) { int i; - // TODO use parse opts + // TODO(bnoordhuis) use parse opts for (i = 1; i < argc; i++) { const char *arg = argv[i]; if (strstr(arg, "--debug") == arg) { @@ -2687,7 +2678,7 @@ static void EnableDebugSignalHandler(uv_signal_t* handle, int) { } -static void RegisterSignalHandler(int signal, void (*handler)(int)) { +static void RegisterSignalHandler(int signal, void (*handler)(int signal)) { struct sigaction sa; memset(&sa, 0, sizeof(sa)); @@ -2713,7 +2704,7 @@ void DebugProcess(const FunctionCallbackInfo& args) { return ThrowErrnoException(errno, "kill"); } } -#endif // __POSIX__ +#endif // __POSIX__ #ifdef _WIN32 @@ -2776,7 +2767,7 @@ static int RegisterDebugSignalHandler() { *handler = EnableDebugThreadProc; - UnmapViewOfFile((void*) handler); + UnmapViewOfFile(static_cast(handler)); return 0; } @@ -2851,20 +2842,16 @@ static void DebugProcess(const FunctionCallbackInfo& args) { } out: - if (process != NULL) { - CloseHandle(process); - } - if (thread != NULL) { + if (process != NULL) + CloseHandle(process); + if (thread != NULL) CloseHandle(thread); - } - if (handler != NULL) { + if (handler != NULL) UnmapViewOfFile(handler); - } - if (mapping != NULL) { + if (mapping != NULL) CloseHandle(mapping); - } } -#endif // _WIN32 +#endif // _WIN32 static void DebugPause(const FunctionCallbackInfo& args) { @@ -2928,7 +2915,7 @@ char** Init(int argc, char *argv[]) { uint32_t *stack_limit = &stack_var - (max_stack_size / sizeof(uint32_t)); constraints.set_stack_limit(stack_limit); - SetResourceConstraints(&constraints); // Must be done before V8::Initialize + SetResourceConstraints(&constraints); // Must be done before V8::Initialize } V8::SetFlagsFromCommandLine(&v8argc, v8argv, false); @@ -2945,7 +2932,7 @@ char** Init(int argc, char *argv[]) { RegisterSignalHandler(SIGPIPE, SIG_IGN); RegisterSignalHandler(SIGINT, SignalExit); RegisterSignalHandler(SIGTERM, SignalExit); -#endif // __POSIX__ +#endif // __POSIX__ uv_check_init(uv_default_loop(), &check_immediate_watcher); uv_unref(reinterpret_cast(&check_immediate_watcher)); @@ -2960,12 +2947,12 @@ char** Init(int argc, char *argv[]) { } else { #ifdef _WIN32 RegisterDebugSignalHandler(); -#else // Posix +#else // Posix static uv_signal_t signal_watcher; uv_signal_init(uv_default_loop(), &signal_watcher); uv_signal_start(&signal_watcher, EnableDebugSignalHandler, SIGUSR1); uv_unref(reinterpret_cast(&signal_watcher)); -#endif // __POSIX__ +#endif // __POSIX__ } return argv; @@ -3018,18 +3005,20 @@ static char **copy_argv(int argc, char **argv) { int i; strlen_sum = 0; - for(i = 0; i < argc; i++) { + for (i = 0; i < argc; i++) { strlen_sum += strlen(argv[i]) + 1; } - argv_copy = (char **) malloc(sizeof(char *) * (argc + 1) + strlen_sum); + argv_copy = static_cast( + malloc(sizeof(*argv_copy) * (argc + 1) + strlen_sum)); if (!argv_copy) { return NULL; } - argv_data = (char *) argv_copy + sizeof(char *) * (argc + 1); + argv_data = reinterpret_cast(argv_copy) + + sizeof(*argv_copy) * (argc + 1); - for(i = 0; i < argc; i++) { + for (i = 0; i < argc; i++) { argv_copy[i] = argv_data; len = strlen(argv[i]) + 1; memcpy(argv_data, argv[i], len); diff --git a/src/node.h b/src/node.h index 7b7c565ef3..12a5d4cbc5 100644 --- a/src/node.h +++ b/src/node.h @@ -139,7 +139,7 @@ void EmitExit(v8::Handle process); // Used to be a macro, hence the uppercase name. template -inline void NODE_SET_METHOD(TypeName& recv, +inline void NODE_SET_METHOD(const TypeName& recv, const char* name, v8::FunctionCallback callback) { v8::Local t = v8::FunctionTemplate::New(callback); @@ -163,7 +163,7 @@ inline void NODE_SET_PROTOTYPE_METHOD(v8::Handle recv, enum encoding {ASCII, UTF8, BASE64, UCS2, BINARY, HEX, BUFFER}; enum encoding ParseEncoding(v8::Handle encoding_v, enum encoding _default = BINARY); -NODE_EXTERN void FatalException(v8::TryCatch &try_catch); +NODE_EXTERN void FatalException(const v8::TryCatch& try_catch); void DisplayExceptionLine(v8::Handle message); NODE_EXTERN v8::Local Encode(const void *buf, size_t len, diff --git a/src/node_buffer.cc b/src/node_buffer.cc index 08790a923c..480b2757e6 100644 --- a/src/node_buffer.cc +++ b/src/node_buffer.cc @@ -53,7 +53,6 @@ size_t length = end - start; namespace node { - namespace Buffer { using v8::Function; @@ -610,7 +609,6 @@ void Initialize(Handle target) { } // namespace Buffer - } // namespace node NODE_MODULE(node_buffer, node::Buffer::Initialize) diff --git a/src/node_buffer.h b/src/node_buffer.h index 37468f2d37..f8db699f79 100644 --- a/src/node_buffer.h +++ b/src/node_buffer.h @@ -23,11 +23,10 @@ #include "smalloc.h" #include "v8.h" -#ifndef NODE_BUFFER_H_ -#define NODE_BUFFER_H_ +#ifndef SRC_NODE_BUFFER_H_ +#define SRC_NODE_BUFFER_H_ namespace node { - namespace Buffer { static const unsigned int kMaxLength = smalloc::kMaxLength; @@ -58,7 +57,6 @@ NODE_EXTERN v8::Local New(char* data, NODE_EXTERN v8::Local Use(char* data, uint32_t len); } // namespace Buffer - } // namespace node -#endif // NODE_BUFFER_H_ +#endif // SRC_NODE_BUFFER_H_ diff --git a/src/node_constants.cc b/src/node_constants.cc index 2aea278cb6..11c67ae519 100644 --- a/src/node_constants.cc +++ b/src/node_constants.cc @@ -38,131 +38,10 @@ namespace node { -using namespace v8; - -void DefineConstants(Handle target) { - // file access modes - NODE_DEFINE_CONSTANT(target, O_RDONLY); - NODE_DEFINE_CONSTANT(target, O_WRONLY); - NODE_DEFINE_CONSTANT(target, O_RDWR); - - NODE_DEFINE_CONSTANT(target, S_IFMT); - NODE_DEFINE_CONSTANT(target, S_IFREG); - NODE_DEFINE_CONSTANT(target, S_IFDIR); - NODE_DEFINE_CONSTANT(target, S_IFCHR); -#ifdef S_IFBLK - NODE_DEFINE_CONSTANT(target, S_IFBLK); -#endif - -#ifdef S_IFIFO - NODE_DEFINE_CONSTANT(target, S_IFIFO); -#endif - -#ifdef S_IFLNK - NODE_DEFINE_CONSTANT(target, S_IFLNK); -#endif - -#ifdef S_IFSOCK - NODE_DEFINE_CONSTANT(target, S_IFSOCK); -#endif - -#ifdef O_CREAT - NODE_DEFINE_CONSTANT(target, O_CREAT); -#endif - -#ifdef O_EXCL - NODE_DEFINE_CONSTANT(target, O_EXCL); -#endif - -#ifdef O_NOCTTY - NODE_DEFINE_CONSTANT(target, O_NOCTTY); -#endif - -#ifdef O_TRUNC - NODE_DEFINE_CONSTANT(target, O_TRUNC); -#endif - -#ifdef O_APPEND - NODE_DEFINE_CONSTANT(target, O_APPEND); -#endif - -#ifdef O_DIRECTORY - NODE_DEFINE_CONSTANT(target, O_DIRECTORY); -#endif - -#ifdef O_EXCL - NODE_DEFINE_CONSTANT(target, O_EXCL); -#endif - -#ifdef O_NOFOLLOW - NODE_DEFINE_CONSTANT(target, O_NOFOLLOW); -#endif - -#ifdef O_SYNC - NODE_DEFINE_CONSTANT(target, O_SYNC); -#endif - -#ifdef O_SYMLINK - NODE_DEFINE_CONSTANT(target, O_SYMLINK); -#endif - -#ifdef O_DIRECT - NODE_DEFINE_CONSTANT(target, O_DIRECT); -#endif - -#ifdef S_IRWXU - NODE_DEFINE_CONSTANT(target, S_IRWXU); -#endif - - -#ifdef S_IRUSR - NODE_DEFINE_CONSTANT(target, S_IRUSR); -#endif - -#ifdef S_IWUSR - NODE_DEFINE_CONSTANT(target, S_IWUSR); -#endif - -#ifdef S_IXUSR - NODE_DEFINE_CONSTANT(target, S_IXUSR); -#endif - - -#ifdef S_IRWXG - NODE_DEFINE_CONSTANT(target, S_IRWXG); -#endif - - -#ifdef S_IRGRP - NODE_DEFINE_CONSTANT(target, S_IRGRP); -#endif - -#ifdef S_IWGRP - NODE_DEFINE_CONSTANT(target, S_IWGRP); -#endif - -#ifdef S_IXGRP - NODE_DEFINE_CONSTANT(target, S_IXGRP); -#endif - - -#ifdef S_IRWXO - NODE_DEFINE_CONSTANT(target, S_IRWXO); -#endif - - -#ifdef S_IROTH - NODE_DEFINE_CONSTANT(target, S_IROTH); -#endif - -#ifdef S_IWOTH - NODE_DEFINE_CONSTANT(target, S_IWOTH); -#endif - -#ifdef S_IXOTH - NODE_DEFINE_CONSTANT(target, S_IXOTH); -#endif +using v8::Handle; +using v8::Object; +void DefineErrnoConstants(Handle target) { #ifdef E2BIG NODE_DEFINE_CONSTANT(target, E2BIG); #endif @@ -478,7 +357,9 @@ void DefineConstants(Handle target) { #ifdef EXDEV NODE_DEFINE_CONSTANT(target, EXDEV); #endif +} +void DefineWindowsErrorConstants(Handle target) { #ifdef WSAEINTR NODE_DEFINE_CONSTANT(target, WSAEINTR); #endif @@ -710,7 +591,9 @@ void DefineConstants(Handle target) { #ifdef WSAEREFUSED NODE_DEFINE_CONSTANT(target, WSAEREFUSED); #endif +} +void DefineSignalConstants(Handle target) { #ifdef SIGHUP NODE_DEFINE_CONSTANT(target, SIGHUP); #endif @@ -853,149 +736,276 @@ void DefineConstants(Handle target) { #ifdef SIGUNUSED NODE_DEFINE_CONSTANT(target, SIGUNUSED); #endif +} -// OpenSSL SSL context options - +void DefineOpenSSLConstants(Handle target) { #ifdef SSL_OP_ALL - NODE_DEFINE_CONSTANT(target, SSL_OP_ALL); + NODE_DEFINE_CONSTANT(target, SSL_OP_ALL); #endif #ifdef SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION - NODE_DEFINE_CONSTANT(target, SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION); + NODE_DEFINE_CONSTANT(target, SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION); #endif #ifdef SSL_OP_CIPHER_SERVER_PREFERENCE - NODE_DEFINE_CONSTANT(target, SSL_OP_CIPHER_SERVER_PREFERENCE); + NODE_DEFINE_CONSTANT(target, SSL_OP_CIPHER_SERVER_PREFERENCE); #endif #ifdef SSL_OP_CISCO_ANYCONNECT - NODE_DEFINE_CONSTANT(target, SSL_OP_CISCO_ANYCONNECT); + NODE_DEFINE_CONSTANT(target, SSL_OP_CISCO_ANYCONNECT); #endif #ifdef SSL_OP_COOKIE_EXCHANGE - NODE_DEFINE_CONSTANT(target, SSL_OP_COOKIE_EXCHANGE); + NODE_DEFINE_CONSTANT(target, SSL_OP_COOKIE_EXCHANGE); #endif #ifdef SSL_OP_CRYPTOPRO_TLSEXT_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_CRYPTOPRO_TLSEXT_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_CRYPTOPRO_TLSEXT_BUG); #endif #ifdef SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS - NODE_DEFINE_CONSTANT(target, SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS); + NODE_DEFINE_CONSTANT(target, SSL_OP_DONT_INSERT_EMPTY_FRAGMENTS); #endif #ifdef SSL_OP_EPHEMERAL_RSA - NODE_DEFINE_CONSTANT(target, SSL_OP_EPHEMERAL_RSA); + NODE_DEFINE_CONSTANT(target, SSL_OP_EPHEMERAL_RSA); #endif #ifdef SSL_OP_LEGACY_SERVER_CONNECT - NODE_DEFINE_CONSTANT(target, SSL_OP_LEGACY_SERVER_CONNECT); + NODE_DEFINE_CONSTANT(target, SSL_OP_LEGACY_SERVER_CONNECT); #endif #ifdef SSL_OP_MICROSOFT_BIG_SSLV3_BUFFER - NODE_DEFINE_CONSTANT(target, SSL_OP_MICROSOFT_BIG_SSLV3_BUFFER); + NODE_DEFINE_CONSTANT(target, SSL_OP_MICROSOFT_BIG_SSLV3_BUFFER); #endif #ifdef SSL_OP_MICROSOFT_SESS_ID_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_MICROSOFT_SESS_ID_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_MICROSOFT_SESS_ID_BUG); #endif #ifdef SSL_OP_MSIE_SSLV2_RSA_PADDING - NODE_DEFINE_CONSTANT(target, SSL_OP_MSIE_SSLV2_RSA_PADDING); + NODE_DEFINE_CONSTANT(target, SSL_OP_MSIE_SSLV2_RSA_PADDING); #endif #ifdef SSL_OP_NETSCAPE_CA_DN_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_CA_DN_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_CA_DN_BUG); #endif #ifdef SSL_OP_NETSCAPE_CHALLENGE_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_CHALLENGE_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_CHALLENGE_BUG); #endif #ifdef SSL_OP_NETSCAPE_DEMO_CIPHER_CHANGE_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_DEMO_CIPHER_CHANGE_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_DEMO_CIPHER_CHANGE_BUG); #endif #ifdef SSL_OP_NETSCAPE_REUSE_CIPHER_CHANGE_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_REUSE_CIPHER_CHANGE_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_NETSCAPE_REUSE_CIPHER_CHANGE_BUG); #endif #ifdef SSL_OP_NO_COMPRESSION - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_COMPRESSION); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_COMPRESSION); #endif #ifdef SSL_OP_NO_QUERY_MTU - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_QUERY_MTU); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_QUERY_MTU); #endif #ifdef SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SESSION_RESUMPTION_ON_RENEGOTIATION); #endif #ifdef SSL_OP_NO_SSLv2 - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SSLv2); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SSLv2); #endif #ifdef SSL_OP_NO_SSLv3 - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SSLv3); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_SSLv3); #endif #ifdef SSL_OP_NO_TICKET - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TICKET); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TICKET); #endif #ifdef SSL_OP_NO_TLSv1 - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1); #endif #ifdef SSL_OP_NO_TLSv1_1 - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1_1); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1_1); #endif #ifdef SSL_OP_NO_TLSv1_2 - NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1_2); + NODE_DEFINE_CONSTANT(target, SSL_OP_NO_TLSv1_2); #endif #ifdef SSL_OP_PKCS1_CHECK_1 - NODE_DEFINE_CONSTANT(target, SSL_OP_PKCS1_CHECK_1); + NODE_DEFINE_CONSTANT(target, SSL_OP_PKCS1_CHECK_1); #endif #ifdef SSL_OP_PKCS1_CHECK_2 - NODE_DEFINE_CONSTANT(target, SSL_OP_PKCS1_CHECK_2); + NODE_DEFINE_CONSTANT(target, SSL_OP_PKCS1_CHECK_2); #endif #ifdef SSL_OP_SINGLE_DH_USE - NODE_DEFINE_CONSTANT(target, SSL_OP_SINGLE_DH_USE); + NODE_DEFINE_CONSTANT(target, SSL_OP_SINGLE_DH_USE); #endif #ifdef SSL_OP_SINGLE_ECDH_USE - NODE_DEFINE_CONSTANT(target, SSL_OP_SINGLE_ECDH_USE); + NODE_DEFINE_CONSTANT(target, SSL_OP_SINGLE_ECDH_USE); #endif #ifdef SSL_OP_SSLEAY_080_CLIENT_DH_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_SSLEAY_080_CLIENT_DH_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_SSLEAY_080_CLIENT_DH_BUG); #endif #ifdef SSL_OP_SSLREF2_REUSE_CERT_TYPE_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_SSLREF2_REUSE_CERT_TYPE_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_SSLREF2_REUSE_CERT_TYPE_BUG); #endif #ifdef SSL_OP_TLS_BLOCK_PADDING_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_BLOCK_PADDING_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_BLOCK_PADDING_BUG); #endif #ifdef SSL_OP_TLS_D5_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_D5_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_D5_BUG); #endif #ifdef SSL_OP_TLS_ROLLBACK_BUG - NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_ROLLBACK_BUG); + NODE_DEFINE_CONSTANT(target, SSL_OP_TLS_ROLLBACK_BUG); #endif #ifdef OPENSSL_NPN_NEGOTIATED #define NPN_ENABLED 1 - NODE_DEFINE_CONSTANT(target, NPN_ENABLED); + NODE_DEFINE_CONSTANT(target, NPN_ENABLED); +#endif +} + +void DefineSystemConstants(Handle target) { + // file access modes + NODE_DEFINE_CONSTANT(target, O_RDONLY); + NODE_DEFINE_CONSTANT(target, O_WRONLY); + NODE_DEFINE_CONSTANT(target, O_RDWR); + + NODE_DEFINE_CONSTANT(target, S_IFMT); + NODE_DEFINE_CONSTANT(target, S_IFREG); + NODE_DEFINE_CONSTANT(target, S_IFDIR); + NODE_DEFINE_CONSTANT(target, S_IFCHR); +#ifdef S_IFBLK + NODE_DEFINE_CONSTANT(target, S_IFBLK); +#endif + +#ifdef S_IFIFO + NODE_DEFINE_CONSTANT(target, S_IFIFO); +#endif + +#ifdef S_IFLNK + NODE_DEFINE_CONSTANT(target, S_IFLNK); +#endif + +#ifdef S_IFSOCK + NODE_DEFINE_CONSTANT(target, S_IFSOCK); +#endif + +#ifdef O_CREAT + NODE_DEFINE_CONSTANT(target, O_CREAT); +#endif + +#ifdef O_EXCL + NODE_DEFINE_CONSTANT(target, O_EXCL); +#endif + +#ifdef O_NOCTTY + NODE_DEFINE_CONSTANT(target, O_NOCTTY); +#endif + +#ifdef O_TRUNC + NODE_DEFINE_CONSTANT(target, O_TRUNC); +#endif + +#ifdef O_APPEND + NODE_DEFINE_CONSTANT(target, O_APPEND); +#endif + +#ifdef O_DIRECTORY + NODE_DEFINE_CONSTANT(target, O_DIRECTORY); +#endif + +#ifdef O_EXCL + NODE_DEFINE_CONSTANT(target, O_EXCL); +#endif + +#ifdef O_NOFOLLOW + NODE_DEFINE_CONSTANT(target, O_NOFOLLOW); +#endif + +#ifdef O_SYNC + NODE_DEFINE_CONSTANT(target, O_SYNC); +#endif + +#ifdef O_SYMLINK + NODE_DEFINE_CONSTANT(target, O_SYMLINK); +#endif + +#ifdef O_DIRECT + NODE_DEFINE_CONSTANT(target, O_DIRECT); +#endif + +#ifdef S_IRWXU + NODE_DEFINE_CONSTANT(target, S_IRWXU); +#endif + +#ifdef S_IRUSR + NODE_DEFINE_CONSTANT(target, S_IRUSR); +#endif + +#ifdef S_IWUSR + NODE_DEFINE_CONSTANT(target, S_IWUSR); +#endif + +#ifdef S_IXUSR + NODE_DEFINE_CONSTANT(target, S_IXUSR); +#endif + +#ifdef S_IRWXG + NODE_DEFINE_CONSTANT(target, S_IRWXG); #endif + +#ifdef S_IRGRP + NODE_DEFINE_CONSTANT(target, S_IRGRP); +#endif + +#ifdef S_IWGRP + NODE_DEFINE_CONSTANT(target, S_IWGRP); +#endif + +#ifdef S_IXGRP + NODE_DEFINE_CONSTANT(target, S_IXGRP); +#endif + +#ifdef S_IRWXO + NODE_DEFINE_CONSTANT(target, S_IRWXO); +#endif + +#ifdef S_IROTH + NODE_DEFINE_CONSTANT(target, S_IROTH); +#endif + +#ifdef S_IWOTH + NODE_DEFINE_CONSTANT(target, S_IWOTH); +#endif + +#ifdef S_IXOTH + NODE_DEFINE_CONSTANT(target, S_IXOTH); +#endif +} + +void DefineConstants(Handle target) { + DefineErrnoConstants(target); + DefineWindowsErrorConstants(target); + DefineSignalConstants(target); + DefineOpenSSLConstants(target); + DefineSystemConstants(target); } } // namespace node diff --git a/src/node_constants.h b/src/node_constants.h index c75f40abf7..f8bc0ce202 100644 --- a/src/node_constants.h +++ b/src/node_constants.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef SRC_CONSTANTS_H_ -#define SRC_CONSTANTS_H_ +#ifndef SRC_NODE_CONSTANTS_H_ +#define SRC_NODE_CONSTANTS_H_ #include "node.h" #include "v8.h" @@ -29,4 +29,4 @@ namespace node { void DefineConstants(v8::Handle target); } // namespace node -#endif // SRC_CONSTANTS_H_ +#endif // SRC_NODE_CONSTANTS_H_ diff --git a/src/node_counters.cc b/src/node_counters.cc index 8f6d136f46..111d282ec7 100644 --- a/src/node_counters.cc +++ b/src/node_counters.cc @@ -86,7 +86,8 @@ static void counter_gc_done(GCType type, GCCallbackFlags flags) { uint64_t gcperiod = endgc - counter_gc_start_time; if (totalperiod > 0) { - unsigned int percent = static_cast((gcperiod * 100) / totalperiod); + unsigned int percent = static_cast( + (gcperiod * 100) / totalperiod); NODE_COUNT_GC_PERCENTTIME(percent); counter_gc_end_time = endgc; @@ -137,4 +138,4 @@ void TermPerfCounters(Handle target) { TermPerfCountersWin32(); } -} +} // namespace node diff --git a/src/node_counters.h b/src/node_counters.h index c55d99b909..1b3d8c2b90 100644 --- a/src/node_counters.h +++ b/src/node_counters.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_COUNTERS_H_ -#define NODE_COUNTERS_H_ +#ifndef SRC_NODE_COUNTERS_H_ +#define SRC_NODE_COUNTERS_H_ #include "node.h" #include "v8.h" @@ -30,7 +30,7 @@ namespace node { void InitPerfCounters(v8::Handle target); void TermPerfCounters(v8::Handle target); -} +} // namespace node #ifdef HAVE_PERFCTR #include "node_win32_perfctr_provider.h" @@ -50,4 +50,4 @@ void TermPerfCounters(v8::Handle target); #define NODE_COUNT_PIPE_BYTES_RECV(bytes) #endif -#endif +#endif // SRC_NODE_COUNTERS_H_ diff --git a/src/node_crypto.cc b/src/node_crypto.cc index ab3b6cc5aa..22b782895c 100644 --- a/src/node_crypto.cc +++ b/src/node_crypto.cc @@ -326,7 +326,7 @@ int SecureContext::NewSessionCallback(SSL* s, SSL_SESSION* sess) { // Takes a string or buffer and loads it into a BIO. // Caller responsible for BIO_free_all-ing the returned object. -static BIO* LoadBIO (Handle v) { +static BIO* LoadBIO(Handle v) { BIO *bio = BIO_new(NodeBIO::GetMethod()); if (!bio) return NULL; @@ -354,8 +354,8 @@ static BIO* LoadBIO (Handle v) { // Takes a string or buffer and loads it into an X509 // Caller responsible for X509_free-ing the returned object. -static X509* LoadX509 (Handle v) { - HandleScope scope(node_isolate); // necessary? +static X509* LoadX509(Handle v) { + HandleScope scope(node_isolate); BIO *bio = LoadBIO(v); if (!bio) return NULL; @@ -467,7 +467,7 @@ int SSL_CTX_use_certificate_chain(SSL_CTX *ctx, BIO *in) { } } -end: + end: if (x != NULL) X509_free(x); return ret; } @@ -648,8 +648,7 @@ void SecureContext::SetSessionIdContext( bio = BIO_new(BIO_s_mem()); if (bio == NULL) { message = String::New("SSL_CTX_set_session_id_context error"); - } - else { + } else { ERR_print_errors(bio); BIO_get_mem_ptr(bio, &mem); message = String::New(mem->data, mem->length); @@ -681,7 +680,7 @@ void SecureContext::Close(const FunctionCallbackInfo& args) { } -//Takes .pfx or .p12 and password in string or buffer format +// Takes .pfx or .p12 and password in string or buffer format void SecureContext::LoadPKCS12(const FunctionCallbackInfo& args) { HandleScope scope(node_isolate); @@ -722,8 +721,7 @@ void SecureContext::LoadPKCS12(const FunctionCallbackInfo& args) { if (d2i_PKCS12_bio(in, &p12) && PKCS12_parse(p12, pass, &pkey, &cert, &extraCerts) && SSL_CTX_use_certificate(sc->ctx_, cert) && - SSL_CTX_use_PrivateKey(sc->ctx_, pkey)) - { + SSL_CTX_use_PrivateKey(sc->ctx_, pkey)) { // set extra certs while (X509* x509 = sk_X509_pop(extraCerts)) { if (!sc->ca_store_) { @@ -817,115 +815,114 @@ size_t ClientHelloParser::Write(const uint8_t* data, size_t len) { Handle argv[1]; switch (state_) { - case kWaiting: - // >= 5 bytes for header parsing - if (offset_ < 5) break; - - if (data_[0] == kChangeCipherSpec || data_[0] == kAlert || - data_[0] == kHandshake || data_[0] == kApplicationData) { - frame_len_ = (data_[3] << 8) + data_[4]; - state_ = kTLSHeader; - body_offset_ = 5; - } else { - frame_len_ = (data_[0] << 8) + data_[1]; - state_ = kSSLHeader; - if (*data_ & 0x40) { - // header with padding - body_offset_ = 3; + case kWaiting: + // >= 5 bytes for header parsing + if (offset_ < 5) + break; + + if (data_[0] == kChangeCipherSpec || + data_[0] == kAlert || + data_[0] == kHandshake || + data_[0] == kApplicationData) { + frame_len_ = (data_[3] << 8) + data_[4]; + state_ = kTLSHeader; + body_offset_ = 5; } else { - // without padding - body_offset_ = 2; + frame_len_ = (data_[0] << 8) + data_[1]; + state_ = kSSLHeader; + if (*data_ & 0x40) { + // header with padding + body_offset_ = 3; + } else { + // without padding + body_offset_ = 2; + } } - } - // Sanity check (too big frame, or too small) - if (frame_len_ >= sizeof(data_)) { - // Let OpenSSL handle it - Finish(); - return copied; - } - case kTLSHeader: - case kSSLHeader: - // >= 5 + frame size bytes for frame parsing - if (offset_ < body_offset_ + frame_len_) break; - - // Skip unsupported frames and gather some data from frame - - // TODO: Check protocol version - if (data_[body_offset_] == kClientHello) { - is_clienthello = true; - uint8_t* body; - size_t session_offset; - - if (state_ == kTLSHeader) { - // Skip frame header, hello header, protocol version and random data - session_offset = body_offset_ + 4 + 2 + 32; - - if (session_offset + 1 < offset_) { - body = data_ + session_offset; - session_size = *body; - session_id = body + 1; - } - } else if (state_ == kSSLHeader) { - // Skip header, version - session_offset = body_offset_ + 3; + // Sanity check (too big frame, or too small) + if (frame_len_ >= sizeof(data_)) { + // Let OpenSSL handle it + Finish(); + return copied; + } + case kTLSHeader: + case kSSLHeader: + // >= 5 + frame size bytes for frame parsing + if (offset_ < body_offset_ + frame_len_) + break; + + // Skip unsupported frames and gather some data from frame + + if (data_[body_offset_] == kClientHello) { + is_clienthello = true; + uint8_t* body; + size_t session_offset; + + if (state_ == kTLSHeader) { + // Skip frame header, hello header, protocol version and random data + session_offset = body_offset_ + 4 + 2 + 32; + + if (session_offset + 1 < offset_) { + body = data_ + session_offset; + session_size = *body; + session_id = body + 1; + } + } else if (state_ == kSSLHeader) { + // Skip header, version + session_offset = body_offset_ + 3; - if (session_offset + 4 < offset_) { - body = data_ + session_offset; + if (session_offset + 4 < offset_) { + body = data_ + session_offset; - int ciphers_size = (body[0] << 8) + body[1]; + int ciphers_size = (body[0] << 8) + body[1]; - if (body + 4 + ciphers_size < data_ + offset_) { - session_size = (body[2] << 8) + body[3]; - session_id = body + 4 + ciphers_size; + if (body + 4 + ciphers_size < data_ + offset_) { + session_size = (body[2] << 8) + body[3]; + session_id = body + 4 + ciphers_size; + } } + } else { + // Whoa? How did we get here? + abort(); + } + + // Check if we overflowed (do not reply with any private data) + if (session_id == NULL || + session_size > 32 || + session_id + session_size > data_ + offset_) { + Finish(); + return copied; } - } else { - // Whoa? How did we get here? - abort(); } - // Check if we overflowed (do not reply with any private data) - if (session_id == NULL || - session_size > 32 || - session_id + session_size > data_ + offset_) { + // Not client hello - let OpenSSL handle it + if (!is_clienthello) { Finish(); return copied; } - // TODO: Parse other things? - } - - // Not client hello - let OpenSSL handle it - if (!is_clienthello) { - Finish(); - return copied; - } - - // Parse frame, call javascript handler and - // move parser into the paused state - if (onclienthello_sym.IsEmpty()) { - onclienthello_sym = String::New("onclienthello"); - } - if (sessionid_sym.IsEmpty()) { - sessionid_sym = String::New("sessionId"); - } - - state_ = kPaused; - hello = Object::New(); - hello->Set(sessionid_sym, - Buffer::New(reinterpret_cast(session_id), - session_size)); - - argv[0] = hello; - MakeCallback(conn_->handle(node_isolate), - onclienthello_sym, - ARRAY_SIZE(argv), - argv); - break; - case kEnded: - default: - break; + // Parse frame, call javascript handler and + // move parser into the paused state + if (onclienthello_sym.IsEmpty()) + onclienthello_sym = String::New("onclienthello"); + if (sessionid_sym.IsEmpty()) + sessionid_sym = String::New("sessionId"); + + state_ = kPaused; + hello = Object::New(); + hello->Set(sessionid_sym, + Buffer::New(reinterpret_cast(session_id), + session_size)); + + argv[0] = hello; + MakeCallback(conn_->handle(node_isolate), + onclienthello_sym, + ARRAY_SIZE(argv), + argv); + break; + case kEnded: + default: + break; } return copied; @@ -954,10 +951,13 @@ int Connection::HandleBIOError(BIO *bio, const char* func, int rv) { if (rv >= 0) return rv; int retry = BIO_should_retry(bio); - (void) retry; // unused if !defined(SSL_PRINT_DEBUG) + (void) retry; // unused if !defined(SSL_PRINT_DEBUG) if (BIO_should_write(bio)) { - DEBUG_PRINT("[%p] BIO: %s want write. should retry %d\n", ssl_, func, retry); + DEBUG_PRINT("[%p] BIO: %s want write. should retry %d\n", + ssl_, + func, + retry); return 0; } else if (BIO_should_read(bio)) { @@ -972,7 +972,11 @@ int Connection::HandleBIOError(BIO *bio, const char* func, int rv) { Local e = Exception::Error(String::New(ssl_error_buf)); handle(node_isolate)->Set(String::New("error"), e); - DEBUG_PRINT("[%p] BIO: %s failed: (%d) %s\n", ssl_, func, rv, ssl_error_buf); + DEBUG_PRINT("[%p] BIO: %s failed: (%d) %s\n", + ssl_, + func, + rv, + ssl_error_buf); return rv; } @@ -1046,7 +1050,7 @@ void Connection::ClearError() { // We should clear the error in JS-land assert( handle(node_isolate)->Get(String::New("error"))->BooleanValue() == false); -#endif // NDEBUG +#endif // NDEBUG } @@ -1079,22 +1083,32 @@ void Connection::Initialize(Handle target) { NODE_SET_PROTOTYPE_METHOD(t, "encOut", Connection::EncOut); NODE_SET_PROTOTYPE_METHOD(t, "clearPending", Connection::ClearPending); NODE_SET_PROTOTYPE_METHOD(t, "encPending", Connection::EncPending); - NODE_SET_PROTOTYPE_METHOD(t, "getPeerCertificate", Connection::GetPeerCertificate); + NODE_SET_PROTOTYPE_METHOD(t, + "getPeerCertificate", + Connection::GetPeerCertificate); NODE_SET_PROTOTYPE_METHOD(t, "getSession", Connection::GetSession); NODE_SET_PROTOTYPE_METHOD(t, "setSession", Connection::SetSession); NODE_SET_PROTOTYPE_METHOD(t, "loadSession", Connection::LoadSession); NODE_SET_PROTOTYPE_METHOD(t, "isSessionReused", Connection::IsSessionReused); NODE_SET_PROTOTYPE_METHOD(t, "isInitFinished", Connection::IsInitFinished); NODE_SET_PROTOTYPE_METHOD(t, "verifyError", Connection::VerifyError); - NODE_SET_PROTOTYPE_METHOD(t, "getCurrentCipher", Connection::GetCurrentCipher); + NODE_SET_PROTOTYPE_METHOD(t, + "getCurrentCipher", + Connection::GetCurrentCipher); NODE_SET_PROTOTYPE_METHOD(t, "start", Connection::Start); NODE_SET_PROTOTYPE_METHOD(t, "shutdown", Connection::Shutdown); - NODE_SET_PROTOTYPE_METHOD(t, "receivedShutdown", Connection::ReceivedShutdown); + NODE_SET_PROTOTYPE_METHOD(t, + "receivedShutdown", + Connection::ReceivedShutdown); NODE_SET_PROTOTYPE_METHOD(t, "close", Connection::Close); #ifdef OPENSSL_NPN_NEGOTIATED - NODE_SET_PROTOTYPE_METHOD(t, "getNegotiatedProtocol", Connection::GetNegotiatedProto); - NODE_SET_PROTOTYPE_METHOD(t, "setNPNProtocols", Connection::SetNPNProtocols); + NODE_SET_PROTOTYPE_METHOD(t, + "getNegotiatedProtocol", + Connection::GetNegotiatedProto); + NODE_SET_PROTOTYPE_METHOD(t, + "setNPNProtocols", + Connection::SetNPNProtocols); #endif @@ -1158,7 +1172,6 @@ int Connection::AdvertiseNextProtoCallback_(SSL *s, const unsigned char** data, unsigned int *len, void *arg) { - Connection *p = static_cast(SSL_get_app_data(s)); if (p->npnProtos_.IsEmpty()) { @@ -1599,8 +1612,8 @@ void Connection::GetPeerCertificate(const FunctionCallbackInfo& args) { EVP_PKEY *pkey = NULL; RSA *rsa = NULL; - if( NULL != (pkey = X509_get_pubkey(peer_cert)) - && NULL != (rsa = EVP_PKEY_get1_RSA(pkey)) ) { + if (NULL != (pkey = X509_get_pubkey(peer_cert)) && + NULL != (rsa = EVP_PKEY_get1_RSA(pkey))) { BN_print(bio, rsa->n); BIO_get_mem_ptr(bio, &mem); info->Set(modulus_symbol, String::New(mem->data, mem->length) ); @@ -1637,7 +1650,7 @@ void Connection::GetPeerCertificate(const FunctionCallbackInfo& args) { const char hex[] = "0123456789ABCDEF"; char fingerprint[EVP_MAX_MD_SIZE * 3]; - for (i = 0; i> 4]; fingerprint[(3*i)+1] = hex[(md[i] & 0x0f)]; fingerprint[(3*i)+2] = ':'; @@ -1645,8 +1658,7 @@ void Connection::GetPeerCertificate(const FunctionCallbackInfo& args) { if (md_size > 0) { fingerprint[(3*(md_size-1))+2] = '\0'; - } - else { + } else { fingerprint[0] = '\0'; } @@ -1840,7 +1852,6 @@ void Connection::VerifyError(const FunctionCallbackInfo& args) { } X509_free(peer_cert); - long x509_verify_error = SSL_get_verify_result(ss->ssl_); Local s; @@ -2678,7 +2689,7 @@ void Sign::SignFinal(const FunctionCallbackInfo& args) { ssize_t len = Buffer::Length(args[0]); char* buf = Buffer::Data(args[0]); - md_len = 8192; // Maximum key size is 8192 bits + md_len = 8192; // Maximum key size is 8192 bits md_value = new unsigned char[md_len]; bool r = sign->SignFinal(&md_value, &md_len, buf, len); @@ -2841,7 +2852,7 @@ bool Verify::VerifyFinal(const char* key_pem, siglen, pkey); -exit: + exit: if (pkey != NULL) EVP_PKEY_free(pkey); if (bp != NULL) @@ -3404,7 +3415,7 @@ void PBKDF2(const FunctionCallbackInfo& args) { } return; -err: + err: delete[] salt; delete[] pass; return ThrowTypeError(type_error); @@ -3414,7 +3425,7 @@ err: struct RandomBytesRequest { ~RandomBytesRequest(); Persistent obj_; - unsigned long error_; // openssl error code or zero + unsigned long error_; // openssl error code or zero uv_work_t work_req_; size_t size_; char* data_; @@ -3460,8 +3471,7 @@ void RandomBytesCheck(RandomBytesRequest* req, Local argv[2]) { argv[0] = Exception::Error(String::New(errmsg)); argv[1] = Null(node_isolate); - } - else { + } else { argv[0] = Null(node_isolate); argv[1] = Buffer::Use(req->data_, req->size_); } @@ -3511,8 +3521,7 @@ void RandomBytes(const FunctionCallbackInfo& args) { RandomBytesWork, RandomBytesAfter); args.GetReturnValue().Set(obj); - } - else { + } else { Local argv[2]; RandomBytesWork(&req->work_req_); RandomBytesCheck(req, argv); @@ -3596,13 +3605,11 @@ void InitCrypto(Handle target) { // Turn off compression. Saves memory - do it in userland. #if !defined(OPENSSL_NO_COMP) - STACK_OF(SSL_COMP)* comp_methods = #if OPENSSL_VERSION_NUMBER < 0x00908000L - SSL_COMP_get_compression_method() + STACK_OF(SSL_COMP)* comp_methods = SSL_COMP_get_compression_method(); #else - SSL_COMP_get_compression_methods() + STACK_OF(SSL_COMP)* comp_methods = SSL_COMP_get_compression_methods(); #endif - ; sk_SSL_COMP_zero(comp_methods); assert(sk_SSL_COMP_num(comp_methods) == 0); #endif diff --git a/src/node_crypto.h b/src/node_crypto.h index a0f16c39ab..aa210df80b 100644 --- a/src/node_crypto.h +++ b/src/node_crypto.h @@ -57,7 +57,7 @@ class SecureContext : ObjectWrap { static void Initialize(v8::Handle target); SSL_CTX *ctx_; - // TODO: ca_store_ should probably be removed, it's not used anywhere. + // TODO(indutny): ca_store_ should probably be removed, it's not used anywhere X509_STORE *ca_store_; static const int kMaxSessionSize = 10 * 1024; @@ -139,10 +139,10 @@ class ClientHelloParser { kEnded }; - ClientHelloParser(Connection* c) : conn_(c), - state_(kWaiting), - offset_(0), - body_offset_(0) { + explicit ClientHelloParser(Connection* c) : conn_(c), + state_(kWaiting), + offset_(0), + body_offset_(0) { } size_t Write(const uint8_t* data, size_t len); @@ -316,9 +316,9 @@ class CipherBase : public ObjectWrap { static void Final(const v8::FunctionCallbackInfo& args); static void SetAutoPadding(const v8::FunctionCallbackInfo& args); - CipherBase(CipherKind kind) : cipher_(NULL), - initialised_(false), - kind_(kind) { + explicit CipherBase(CipherKind kind) : cipher_(NULL), + initialised_(false), + kind_(kind) { } ~CipherBase() { @@ -335,7 +335,7 @@ class CipherBase : public ObjectWrap { class Hmac : public ObjectWrap { public: - static void Initialize (v8::Handle target); + static void Initialize(v8::Handle target); protected: void HmacInit(const char* hash_type, const char* key, int key_len); @@ -363,7 +363,7 @@ class Hmac : public ObjectWrap { class Hash : public ObjectWrap { public: - static void Initialize (v8::Handle target); + static void Initialize(v8::Handle target); bool HashInit(const char* hash_type); bool HashUpdate(const char* data, int len); @@ -420,7 +420,7 @@ class Sign : public ObjectWrap { class Verify : public ObjectWrap { public: - static void Initialize (v8::Handle target); + static void Initialize(v8::Handle target); void VerifyInit(const char* verify_type); bool VerifyUpdate(const char* data, int len); @@ -430,7 +430,7 @@ class Verify : public ObjectWrap { int siglen); protected: - static void New (const v8::FunctionCallbackInfo& args); + static void New(const v8::FunctionCallbackInfo& args); static void VerifyInit(const v8::FunctionCallbackInfo& args); static void VerifyUpdate(const v8::FunctionCallbackInfo& args); static void VerifyFinal(const v8::FunctionCallbackInfo& args); @@ -447,7 +447,6 @@ class Verify : public ObjectWrap { EVP_MD_CTX mdctx_; /* coverity[member_decl] */ const EVP_MD* md_; /* coverity[member_decl] */ bool initialised_; - }; class DiffieHellman : public ObjectWrap { diff --git a/src/node_crypto_bio.cc b/src/node_crypto_bio.cc index 2932f64bf2..6783184ed1 100644 --- a/src/node_crypto_bio.cc +++ b/src/node_crypto_bio.cc @@ -133,49 +133,49 @@ long NodeBIO::Ctrl(BIO* bio, int cmd, long num, void* ptr) { ret = 1; switch (cmd) { - case BIO_CTRL_RESET: - nbio->Reset(); - break; - case BIO_CTRL_EOF: - ret = nbio->Length() == 0; - break; - case BIO_C_SET_BUF_MEM_EOF_RETURN: - bio->num = num; - break; - case BIO_CTRL_INFO: - ret = nbio->Length(); - if (ptr != NULL) - *reinterpret_cast(ptr) = NULL; - break; - case BIO_C_SET_BUF_MEM: - assert(0 && "Can't use SET_BUF_MEM_PTR with NodeBIO"); - abort(); - break; - case BIO_C_GET_BUF_MEM_PTR: - assert(0 && "Can't use GET_BUF_MEM_PTR with NodeBIO"); - ret = 0; - break; - case BIO_CTRL_GET_CLOSE: - ret = bio->shutdown; - break; - case BIO_CTRL_SET_CLOSE: - bio->shutdown = num; - break; - case BIO_CTRL_WPENDING: - ret = 0; - break; - case BIO_CTRL_PENDING: - ret = nbio->Length(); - break; - case BIO_CTRL_DUP: - case BIO_CTRL_FLUSH: - ret = 1; - break; - case BIO_CTRL_PUSH: - case BIO_CTRL_POP: - default: - ret = 0; - break; + case BIO_CTRL_RESET: + nbio->Reset(); + break; + case BIO_CTRL_EOF: + ret = nbio->Length() == 0; + break; + case BIO_C_SET_BUF_MEM_EOF_RETURN: + bio->num = num; + break; + case BIO_CTRL_INFO: + ret = nbio->Length(); + if (ptr != NULL) + *reinterpret_cast(ptr) = NULL; + break; + case BIO_C_SET_BUF_MEM: + assert(0 && "Can't use SET_BUF_MEM_PTR with NodeBIO"); + abort(); + break; + case BIO_C_GET_BUF_MEM_PTR: + assert(0 && "Can't use GET_BUF_MEM_PTR with NodeBIO"); + ret = 0; + break; + case BIO_CTRL_GET_CLOSE: + ret = bio->shutdown; + break; + case BIO_CTRL_SET_CLOSE: + bio->shutdown = num; + break; + case BIO_CTRL_WPENDING: + ret = 0; + break; + case BIO_CTRL_PENDING: + ret = nbio->Length(); + break; + case BIO_CTRL_DUP: + case BIO_CTRL_FLUSH: + ret = 1; + break; + case BIO_CTRL_PUSH: + case BIO_CTRL_POP: + default: + ret = 0; + break; } return ret; } @@ -387,4 +387,4 @@ NodeBIO::~NodeBIO() { write_head_ = NULL; } -} // namespace node +} // namespace node diff --git a/src/node_crypto_bio.h b/src/node_crypto_bio.h index dc45faee2d..906489fcc9 100644 --- a/src/node_crypto_bio.h +++ b/src/node_crypto_bio.h @@ -19,6 +19,9 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. +#ifndef SRC_NODE_CRYPTO_BIO_H_ +#define SRC_NODE_CRYPTO_BIO_H_ + #include "openssl/bio.h" #include @@ -110,4 +113,6 @@ class NodeBIO { static BIO_METHOD method_; }; -} // namespace node +} // namespace node + +#endif // SRC_NODE_CRYPTO_BIO_H_ diff --git a/src/node_crypto_groups.h b/src/node_crypto_groups.h index 86ddd2ddaf..7ef0d8dc8e 100644 --- a/src/node_crypto_groups.h +++ b/src/node_crypto_groups.h @@ -1,3 +1,26 @@ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +#ifndef SRC_NODE_CRYPTO_GROUPS_H_ +#define SRC_NODE_CRYPTO_GROUPS_H_ /* These modular groups were literally taken from: @@ -385,3 +408,5 @@ static const modp_group modp_groups[] = { { "modp18", V(group_modp18), sizeof(group_modp18), V(two_generator), 1 } #undef V }; + +#endif // SRC_NODE_CRYPTO_GROUPS_H_ diff --git a/src/node_dtrace.cc b/src/node_dtrace.cc index b7b0822c37..9d7048be3d 100644 --- a/src/node_dtrace.cc +++ b/src/node_dtrace.cc @@ -347,4 +347,4 @@ void InitDTrace(Handle target) { #endif } -} +} // namespace node diff --git a/src/node_dtrace.h b/src/node_dtrace.h index 5374ab60c0..5b6df8f4a7 100644 --- a/src/node_dtrace.h +++ b/src/node_dtrace.h @@ -19,14 +19,13 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_DTRACE_H_ -#define NODE_DTRACE_H_ +#ifndef SRC_NODE_DTRACE_H_ +#define SRC_NODE_DTRACE_H_ #include "node.h" #include "v8.h" extern "C" { - /* * The following structures are passed directly to DTrace when probes are fired. * Translators in node.d translate these structures into the corresponding D @@ -36,15 +35,15 @@ extern "C" { */ typedef struct { - int32_t fd; - int32_t port; - char *remote; - int32_t buffered; + int32_t fd; + int32_t port; + char* remote; + int32_t buffered; } node_dtrace_connection_t; typedef struct { - char *url; - char *method; + char* url; + char* method; } node_dtrace_http_client_request_t; /* @@ -61,22 +60,22 @@ typedef struct { * a field is present by checking it against NULL. */ typedef struct { - union { - uint32_t version; - uintptr_t unused; /* for compat. with old 64-bit struct */ - } _un; - char *url; - char *method; - char *forwardedFor; - char *_pad[8]; + union { + uint32_t version; + uintptr_t unused; /* for compat. with old 64-bit struct */ + } _un; + char* url; + char* method; + char* forwardedFor; + char* _pad[8]; } node_dtrace_http_server_request_t; -} +} // extern "C" namespace node { void InitDTrace(v8::Handle target); -} +} // namespace node -#endif +#endif // SRC_NODE_DTRACE_H_ diff --git a/src/node_extensions.cc b/src/node_extensions.cc index bbd8111edb..a4a2cab266 100644 --- a/src/node_extensions.cc +++ b/src/node_extensions.cc @@ -22,6 +22,7 @@ #include "node.h" #include "node_version.h" +#include "node_extensions.h" #include #include @@ -33,7 +34,7 @@ #define NODE_EXT_LIST_ITEM NODE_MODULE_DECL #define NODE_EXT_LIST_END -#include "node_extensions.h" +NODE_EXT_LIST(NODE_EXT_LIST_START, NODE_EXT_LIST_ITEM, NODE_EXT_LIST_END) #undef NODE_EXT_LIST_START #undef NODE_EXT_LIST_ITEM @@ -44,12 +45,11 @@ #define NODE_EXT_LIST_ITEM NODE_EXT_STRING #define NODE_EXT_LIST_END NULL}; -#include "node_extensions.h" +NODE_EXT_LIST(NODE_EXT_LIST_START, NODE_EXT_LIST_ITEM, NODE_EXT_LIST_END) namespace node { -node_module_struct* get_builtin_module(const char *name) -{ +node_module_struct* get_builtin_module(const char *name) { char buf[128]; node_module_struct *cur = NULL; snprintf(buf, sizeof(buf), "node_%s", name); @@ -65,4 +65,4 @@ node_module_struct* get_builtin_module(const char *name) return NULL; } -}; // namespace node +} // namespace node diff --git a/src/node_extensions.h b/src/node_extensions.h index fd0d0d1fc2..37beacc51a 100644 --- a/src/node_extensions.h +++ b/src/node_extensions.h @@ -19,28 +19,39 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -NODE_EXT_LIST_START -NODE_EXT_LIST_ITEM(node_buffer) -#if HAVE_OPENSSL -NODE_EXT_LIST_ITEM(node_crypto) -NODE_EXT_LIST_ITEM(node_tls_wrap) -#endif -NODE_EXT_LIST_ITEM(node_evals) -NODE_EXT_LIST_ITEM(node_fs) -NODE_EXT_LIST_ITEM(node_http_parser) -NODE_EXT_LIST_ITEM(node_os) -NODE_EXT_LIST_ITEM(node_smalloc) -NODE_EXT_LIST_ITEM(node_zlib) +#ifndef SRC_NODE_EXTENSIONS_H_ +#define SRC_NODE_EXTENSIONS_H_ -NODE_EXT_LIST_ITEM(node_uv) -NODE_EXT_LIST_ITEM(node_timer_wrap) -NODE_EXT_LIST_ITEM(node_tcp_wrap) -NODE_EXT_LIST_ITEM(node_udp_wrap) -NODE_EXT_LIST_ITEM(node_pipe_wrap) -NODE_EXT_LIST_ITEM(node_cares_wrap) -NODE_EXT_LIST_ITEM(node_tty_wrap) -NODE_EXT_LIST_ITEM(node_process_wrap) -NODE_EXT_LIST_ITEM(node_fs_event_wrap) -NODE_EXT_LIST_ITEM(node_signal_wrap) +#ifdef HAVE_OPENSSL +# define NODE_EXT_LIST_SSL(ITEM) \ + ITEM(node_crypto) \ + ITEM(node_tls_wrap) +#else +# define NODE_EXT_LIST_SSL(ITEM) +#endif // HAVE_OPENSSL -NODE_EXT_LIST_END +#define NODE_EXT_LIST(START, ITEM, END) \ + START \ + ITEM(node_buffer) \ + NODE_EXT_LIST_SSL(ITEM) \ + ITEM(node_evals) \ + ITEM(node_fs) \ + ITEM(node_http_parser) \ + ITEM(node_os) \ + ITEM(node_smalloc) \ + ITEM(node_zlib) \ + \ + ITEM(node_uv) \ + ITEM(node_timer_wrap) \ + ITEM(node_tcp_wrap) \ + ITEM(node_udp_wrap) \ + ITEM(node_pipe_wrap) \ + ITEM(node_cares_wrap) \ + ITEM(node_tty_wrap) \ + ITEM(node_process_wrap) \ + ITEM(node_fs_event_wrap) \ + ITEM(node_signal_wrap) \ + \ + END \ + +#endif // SRC_NODE_EXTENSIONS_H_ diff --git a/src/node_file.cc b/src/node_file.cc index 1390ebcdbe..403ec10c94 100644 --- a/src/node_file.cc +++ b/src/node_file.cc @@ -55,7 +55,7 @@ using v8::Persistent; using v8::String; using v8::Value; -#define MIN(a,b) ((a) < (b) ? (a) : (b)) +#define MIN(a, b) ((a) < (b) ? (a) : (b)) #define TYPE_ERROR(msg) ThrowTypeError(msg) @@ -63,13 +63,13 @@ using v8::Value; class FSReqWrap: public ReqWrap { public: - FSReqWrap(const char* syscall) + explicit FSReqWrap(const char* syscall) : must_free_(false), syscall_(syscall) { } const char* syscall() { return syscall_; } - bool must_free_; // request is responsible for free'ing memory oncomplete + bool must_free_; // request is responsible for free'ing memory oncomplete private: const char* syscall_; @@ -98,7 +98,7 @@ static inline int IsInt64(double x) { static void After(uv_fs_t *req) { HandleScope scope(node_isolate); - FSReqWrap* req_wrap = (FSReqWrap*) req->data; + FSReqWrap* req_wrap = reinterpret_cast(req->data); assert(&req_wrap->req_ == req); // check if data needs to be cleaned @@ -833,13 +833,13 @@ static void Read(const FunctionCallbackInfo& args) { static void Chmod(const FunctionCallbackInfo& args) { HandleScope scope(node_isolate); - if(args.Length() < 2 || !args[0]->IsString() || !args[1]->IsInt32()) { + if (args.Length() < 2 || !args[0]->IsString() || !args[1]->IsInt32()) { return THROW_BAD_ARGS; } String::Utf8Value path(args[0]); int mode = static_cast(args[1]->Int32Value()); - if(args[2]->IsFunction()) { + if (args[2]->IsFunction()) { ASYNC_CALL(chmod, args[2], *path, mode); } else { SYNC_CALL(chmod, *path, *path, mode); @@ -853,13 +853,13 @@ static void Chmod(const FunctionCallbackInfo& args) { static void FChmod(const FunctionCallbackInfo& args) { HandleScope scope(node_isolate); - if(args.Length() < 2 || !args[0]->IsInt32() || !args[1]->IsInt32()) { + if (args.Length() < 2 || !args[0]->IsInt32() || !args[1]->IsInt32()) { return THROW_BAD_ARGS; } int fd = args[0]->Int32Value(); int mode = static_cast(args[1]->Int32Value()); - if(args[2]->IsFunction()) { + if (args[2]->IsFunction()) { ASYNC_CALL(fchmod, args[2], fd, mode); } else { SYNC_CALL(fchmod, 0, fd, mode); @@ -989,11 +989,11 @@ void File::Initialize(Handle target) { NODE_SET_METHOD(target, "chmod", Chmod); NODE_SET_METHOD(target, "fchmod", FChmod); - //NODE_SET_METHOD(target, "lchmod", LChmod); + // NODE_SET_METHOD(target, "lchmod", LChmod); NODE_SET_METHOD(target, "chown", Chown); NODE_SET_METHOD(target, "fchown", FChown); - //NODE_SET_METHOD(target, "lchown", LChown); + // NODE_SET_METHOD(target, "lchown", LChown); NODE_SET_METHOD(target, "utimes", UTimes); NODE_SET_METHOD(target, "futimes", FUTimes); diff --git a/src/node_file.h b/src/node_file.h index 5757e732b5..0a5d270101 100644 --- a/src/node_file.h +++ b/src/node_file.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef SRC_FILE_H_ -#define SRC_FILE_H_ +#ifndef SRC_NODE_FILE_H_ +#define SRC_NODE_FILE_H_ #include "node.h" #include "v8.h" @@ -35,4 +35,5 @@ class File { void InitFs(v8::Handle target); } // namespace node -#endif // SRC_FILE_H_ + +#endif // SRC_NODE_FILE_H_ diff --git a/src/node_http_parser.cc b/src/node_http_parser.cc index de89b4e74e..2485b0acbf 100644 --- a/src/node_http_parser.cc +++ b/src/node_http_parser.cc @@ -161,7 +161,7 @@ struct StringPtr { str_ = str; else if (on_heap_ || str_ + size_ != str) { // Non-consecutive input, make a copy on the heap. - // TODO Use slab allocation, O(n) allocs is bad. + // TODO(bnoordhuis) Use slab allocation, O(n) allocs is bad. char* s = new char[size_ + size]; memcpy(s, str_, size_); memcpy(s + size_, str, size); @@ -192,8 +192,8 @@ struct StringPtr { class Parser : public ObjectWrap { -public: - Parser(enum http_parser_type type) : ObjectWrap() { + public: + explicit Parser(enum http_parser_type type) : ObjectWrap() { Init(type); } @@ -228,7 +228,7 @@ public: fields_[num_fields_ - 1].Reset(); } - assert(num_fields_ < (int)ARRAY_SIZE(fields_)); + assert(num_fields_ < static_cast(ARRAY_SIZE(fields_))); assert(num_fields_ == num_values_ + 1); fields_[num_fields_ - 1].Update(at, length); @@ -244,7 +244,7 @@ public: values_[num_values_ - 1].Reset(); } - assert(num_values_ < (int)ARRAY_SIZE(values_)); + assert(num_values_ < static_cast(ARRAY_SIZE(values_))); assert(num_values_ == num_fields_); values_[num_values_ - 1].Update(at, length); @@ -265,8 +265,7 @@ public: if (have_flushed_) { // Slow case, flush remaining headers. Flush(); - } - else { + } else { // Fast case, pass headers and URL to JS land. message_info->Set(headers_sym, CreateHeaders()); if (parser_.type == HTTP_REQUEST) @@ -342,7 +341,7 @@ public: HandleScope scope(node_isolate); if (num_fields_) - Flush(); // Flush trailing HTTP headers. + Flush(); // Flush trailing HTTP headers. Local obj = handle(node_isolate); Local cb = obj->Get(on_message_complete_sym); @@ -430,7 +429,7 @@ public: Local nparsed_obj = Integer::New(nparsed, node_isolate); // If there was a parse error in one of the callbacks - // TODO What if there is an error on EOF? + // TODO(bnoordhuis) What if there is an error on EOF? if (!parser->parser_.upgrade && nparsed != buffer_len) { enum http_errno err = HTTP_PARSER_ERRNO(&parser->parser_); @@ -481,7 +480,7 @@ public: } -private: + private: Local CreateHeaders() { // num_values_ is either -1 or the entry # of the last header diff --git a/src/node_http_parser.h b/src/node_http_parser.h index d720b3f223..e376f52145 100644 --- a/src/node_http_parser.h +++ b/src/node_http_parser.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_HTTP_PARSER -#define NODE_HTTP_PARSER +#ifndef SRC_NODE_HTTP_PARSER_H_ +#define SRC_NODE_HTTP_PARSER_H_ #include "v8.h" @@ -30,6 +30,6 @@ namespace node { void InitHttpParser(v8::Handle target); -} +} // namespace node -#endif // NODE_HTTP_PARSER +#endif // SRC_NODE_HTTP_PARSER_H_ diff --git a/src/node_internals.h b/src/node_internals.h index 03d6d57afb..86c486714a 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -39,12 +39,12 @@ extern v8::Persistent process_p; template class CachedBase { -public: + public: CachedBase(); operator v8::Handle() const; void operator=(v8::Handle that); // Can only assign once. bool IsEmpty() const; -private: + private: CachedBase(const CachedBase&); void operator=(const CachedBase&); v8::Persistent handle_; @@ -52,14 +52,14 @@ private: template class Cached : public CachedBase { -public: + public: operator v8::Handle() const; void operator=(v8::Handle that); }; template <> class Cached : public CachedBase { -public: + public: operator v8::Handle() const; void operator=(v8::Handle that); }; @@ -93,12 +93,14 @@ v8::Handle MakeCallback( int argc, v8::Handle* argv); -inline bool HasInstance(v8::Persistent& function_template, - v8::Handle value); +inline bool HasInstance( + const v8::Persistent& function_template, + v8::Handle value); -inline v8::Local NewInstance(v8::Persistent& ctor, - int argc = 0, - v8::Handle* argv = NULL); +inline v8::Local NewInstance( + const v8::Persistent& ctor, + int argc = 0, + v8::Handle* argv = NULL); // Convert a struct sockaddr to a { address: '1.2.3.4', port: 1234 } JS object. // Sets address and port properties on the info object and returns it. @@ -131,12 +133,14 @@ inline static int snprintf(char* buf, unsigned int len, const char* fmt, ...) { // g++ in strict mode complains loudly about the system offsetof() macro // because it uses NULL as the base address. # define offset_of(type, member) \ - ((intptr_t) ((char *) &(((type *) 8)->member) - 8)) + (reinterpret_cast( \ + reinterpret_cast(&(reinterpret_cast(8)->member)) - 8)) #endif #ifndef container_of # define container_of(ptr, type, member) \ - ((type *) ((char *) (ptr) - offset_of(type, member))) + (reinterpret_cast(reinterpret_cast(ptr) - \ + offset_of(type, member))) #endif #ifndef ARRAY_SIZE @@ -324,20 +328,22 @@ v8::Handle MakeCallback( return MakeCallback(recv, handle, argc, argv); } -inline bool HasInstance(v8::Persistent& function_template, - v8::Handle value) { +inline bool HasInstance( + const v8::Persistent& function_template, + v8::Handle value) { v8::Local function_template_handle = PersistentToLocal(function_template); return function_template_handle->HasInstance(value); } -inline v8::Local NewInstance(v8::Persistent& ctor, - int argc, - v8::Handle* argv) { +inline v8::Local NewInstance( + const v8::Persistent& ctor, + int argc, + v8::Handle* argv) { v8::Local constructor_handle = PersistentToLocal(ctor); return constructor_handle->NewInstance(argc, argv); } -} // namespace node +} // namespace node -#endif // SRC_NODE_INTERNALS_H_ +#endif // SRC_NODE_INTERNALS_H_ diff --git a/src/node_javascript.cc b/src/node_javascript.cc index d9adecb70d..76e3dfaea6 100644 --- a/src/node_javascript.cc +++ b/src/node_javascript.cc @@ -28,22 +28,26 @@ #include #endif -using namespace v8; - namespace node { +using v8::Handle; +using v8::HandleScope; +using v8::Local; +using v8::Object; +using v8::String; + Handle MainSource() { return String::New(node_native, sizeof(node_native) - 1); } -void DefineJavaScript(v8::Handle target) { +void DefineJavaScript(Handle target) { HandleScope scope(node_isolate); for (int i = 0; natives[i].name; i++) { if (natives[i].source != node_native) { Local name = String::New(natives[i].name); Handle source = String::New(natives[i].source, - natives[i].source_len); + natives[i].source_len); target->Set(name, source); } } diff --git a/src/node_javascript.h b/src/node_javascript.h index 2cfa979547..5193ea5350 100644 --- a/src/node_javascript.h +++ b/src/node_javascript.h @@ -19,6 +19,9 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. +#ifndef SRC_NODE_JAVASCRIPT_H_ +#define SRC_NODE_JAVASCRIPT_H_ + #include "v8.h" namespace node { @@ -27,3 +30,5 @@ void DefineJavaScript(v8::Handle target); v8::Handle MainSource(); } // namespace node + +#endif // SRC_NODE_JAVASCRIPT_H_ diff --git a/src/node_object_wrap.h b/src/node_object_wrap.h index 990c82b1db..5c6ab2865b 100644 --- a/src/node_object_wrap.h +++ b/src/node_object_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef object_wrap_h -#define object_wrap_h +#ifndef SRC_NODE_OBJECT_WRAP_H_ +#define SRC_NODE_OBJECT_WRAP_H_ #include "node.h" #include "v8.h" @@ -38,7 +38,7 @@ namespace node { class NODE_EXTERN ObjectWrap { -public: + public: ObjectWrap() { refs_ = 0; } @@ -75,7 +75,7 @@ public: } -protected: + protected: inline void Wrap(v8::Handle handle) { assert(persistent().IsEmpty()); assert(handle->InternalFieldCount() > 0); @@ -118,7 +118,7 @@ protected: int refs_; // ro -private: + private: static void WeakCallback(v8::Isolate* isolate, v8::Persistent* pobj, ObjectWrap* wrap) { @@ -132,6 +132,6 @@ private: v8::Persistent handle_; }; -} // namespace node +} // namespace node -#endif // object_wrap_h +#endif // SRC_NODE_OBJECT_WRAP_H_ diff --git a/src/node_os.cc b/src/node_os.cc index 224701a76b..d668c16d39 100644 --- a/src/node_os.cc +++ b/src/node_os.cc @@ -30,19 +30,19 @@ #ifdef __MINGW32__ # include -#endif +#endif // __MINGW32__ #ifdef __POSIX__ # include // MAXHOSTNAMELEN on Solaris. # include // gethostname, sysconf # include // MAXHOSTNAMELEN on Linux and the BSDs. # include -#endif +#endif // __MINGW32__ // Add Windows fallback. #ifndef MAXHOSTNAMELEN # define MAXHOSTNAMELEN 256 -#endif +#endif // MAXHOSTNAMELEN namespace node { @@ -70,9 +70,9 @@ static void GetHostname(const FunctionCallbackInfo& args) { if (gethostname(buf, sizeof(buf))) { #ifdef __POSIX__ int errorno = errno; -#else // __MINGW32__ +#else // __MINGW32__ int errorno = WSAGetLastError(); -#endif // __MINGW32__ +#endif // __POSIX__ return ThrowErrnoException(errorno, "gethostname"); } buf[sizeof(buf) - 1] = '\0'; @@ -91,9 +91,9 @@ static void GetOSType(const FunctionCallbackInfo& args) { return ThrowErrnoException(errno, "uname"); } rval = info.sysname; -#else // __MINGW32__ +#else // __MINGW32__ rval ="Windows_NT"; -#endif +#endif // __POSIX__ args.GetReturnValue().Set(String::New(rval)); } @@ -109,17 +109,21 @@ static void GetOSRelease(const FunctionCallbackInfo& args) { return ThrowErrnoException(errno, "uname"); } rval = info.release; -#else // __MINGW32__ +#else // __MINGW32__ char release[256]; OSVERSIONINFO info; info.dwOSVersionInfoSize = sizeof(info); if (GetVersionEx(&info) == 0) return; - sprintf(release, "%d.%d.%d", static_cast(info.dwMajorVersion), - static_cast(info.dwMinorVersion), static_cast(info.dwBuildNumber)); + snprintf(release, + sizeof(release), + "%d.%d.%d", + static_cast(info.dwMajorVersion), + static_cast(info.dwMinorVersion), + static_cast(info.dwBuildNumber)); rval = release; -#endif +#endif // __POSIX__ args.GetReturnValue().Set(String::New(rval)); } @@ -225,7 +229,7 @@ static void GetInterfaceAddresses(const FunctionCallbackInfo& args) { } if (interfaces[i].address.address4.sin_family == AF_INET) { - uv_ip4_name(&interfaces[i].address.address4,ip, sizeof(ip)); + uv_ip4_name(&interfaces[i].address.address4, ip, sizeof(ip)); uv_ip4_name(&interfaces[i].netmask.netmask4, netmask, sizeof(netmask)); family = String::New("IPv4"); } else if (interfaces[i].address.address4.sin_family == AF_INET6) { diff --git a/src/node_os.h b/src/node_os.h index 25ff71eee3..f4968635d0 100644 --- a/src/node_os.h +++ b/src/node_os.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef node_os_h -#define node_os_h +#ifndef SRC_NODE_OS_H_ +#define SRC_NODE_OS_H_ #include "node.h" #include "v8.h" @@ -28,11 +28,10 @@ namespace node { class OS { -public: - static void Initialize (v8::Handle target); + public: + static void Initialize(v8::Handle target); }; - } // namespace node -#endif // node_os_h +#endif // SRC_NODE_OS_H_ diff --git a/src/node_script.cc b/src/node_script.cc index 30d20022f1..5cf9189d85 100644 --- a/src/node_script.cc +++ b/src/node_script.cc @@ -310,9 +310,8 @@ void WrappedScript::EvalMachine(const FunctionCallbackInfo& args) { } const int sandbox_index = input_flag == compileCode ? 1 : 0; - if (context_flag == userContext - && !WrappedContext::InstanceOf(args[sandbox_index])) - { + if (context_flag == userContext && + !WrappedContext::InstanceOf(args[sandbox_index])) { return ThrowTypeError("needs a 'context' argument."); } diff --git a/src/node_script.h b/src/node_script.h index 047829f539..5914f56dfd 100644 --- a/src/node_script.h +++ b/src/node_script.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef node_script_h -#define node_script_h +#ifndef SRC_NODE_SCRIPT_H_ +#define SRC_NODE_SCRIPT_H_ #include "node.h" #include "node_object_wrap.h" @@ -29,8 +29,8 @@ namespace node { - void InitEvals(v8::Handle target); -} // namespace node -#endif // node_script_h +} // namespace node + +#endif // SRC_NODE_SCRIPT_H_ diff --git a/src/node_stat_watcher.cc b/src/node_stat_watcher.cc index 18dd34a0e3..96e8c192a6 100644 --- a/src/node_stat_watcher.cc +++ b/src/node_stat_watcher.cc @@ -60,10 +60,8 @@ static void Delete(uv_handle_t* handle) { } -StatWatcher::StatWatcher() - : ObjectWrap() - , watcher_(new uv_fs_poll_t) -{ +StatWatcher::StatWatcher() : ObjectWrap(), + watcher_(new uv_fs_poll_t) { uv_fs_poll_init(uv_default_loop(), watcher_); watcher_->data = static_cast(this); } @@ -130,7 +128,7 @@ void StatWatcher::Stop(const FunctionCallbackInfo& args) { } -void StatWatcher::Stop () { +void StatWatcher::Stop() { if (!uv_is_active(reinterpret_cast(watcher_))) return; uv_fs_poll_stop(watcher_); Unref(); diff --git a/src/node_stat_watcher.h b/src/node_stat_watcher.h index 9fd5820614..cb5f091f16 100644 --- a/src/node_stat_watcher.h +++ b/src/node_stat_watcher.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_STAT_WATCHER_H_ -#define NODE_STAT_WATCHER_H_ +#ifndef SRC_NODE_STAT_WATCHER_H_ +#define SRC_NODE_STAT_WATCHER_H_ #include "node.h" #include "uv.h" @@ -52,4 +52,4 @@ class StatWatcher : ObjectWrap { }; } // namespace node -#endif // NODE_STAT_WATCHER_H_ +#endif // SRC_NODE_STAT_WATCHER_H_ diff --git a/src/node_version.h b/src/node_version.h index cf069aba88..cf3d6c07b7 100644 --- a/src/node_version.h +++ b/src/node_version.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_VERSION_H -#define NODE_VERSION_H +#ifndef SRC_NODE_VERSION_H_ +#define SRC_NODE_VERSION_H_ #define NODE_MAJOR_VERSION 0 #define NODE_MINOR_VERSION 11 @@ -55,7 +55,8 @@ #define NODE_VERSION_AT_LEAST(major, minor, patch) \ (( (major) < NODE_MAJOR_VERSION) \ || ((major) == NODE_MAJOR_VERSION && (minor) < NODE_MINOR_VERSION) \ - || ((major) == NODE_MAJOR_VERSION && (minor) == NODE_MINOR_VERSION && (patch) <= NODE_PATCH_VERSION)) + || ((major) == NODE_MAJOR_VERSION && \ + (minor) == NODE_MINOR_VERSION && (patch) <= NODE_PATCH_VERSION)) /** * When this version number is changed, node.js will refuse @@ -65,4 +66,4 @@ */ #define NODE_MODULE_VERSION 0x000C /* v0.12 */ -#endif /* NODE_VERSION_H */ +#endif /* SRC_NODE_VERSION_H_ */ diff --git a/src/node_watchdog.cc b/src/node_watchdog.cc index 0922a2b482..32c36ce1b5 100644 --- a/src/node_watchdog.cc +++ b/src/node_watchdog.cc @@ -27,10 +27,7 @@ namespace node { using v8::V8; -Watchdog::Watchdog(uint64_t ms) - : thread_created_(false) - , destroyed_(false) { - +Watchdog::Watchdog(uint64_t ms) : thread_created_(false), destroyed_(false) { loop_ = uv_loop_new(); if (!loop_) return; diff --git a/src/node_watchdog.h b/src/node_watchdog.h index 92a8081047..9ac82cf958 100644 --- a/src/node_watchdog.h +++ b/src/node_watchdog.h @@ -29,7 +29,7 @@ namespace node { class Watchdog { public: - Watchdog(uint64_t ms); + explicit Watchdog(uint64_t ms); ~Watchdog(); void Dispose(); diff --git a/src/node_win32_etw_provider-inl.h b/src/node_win32_etw_provider-inl.h index 1c4a2431c6..17d50a8206 100644 --- a/src/node_win32_etw_provider-inl.h +++ b/src/node_win32_etw_provider-inl.h @@ -19,16 +19,14 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef SRC_ETW_INL_H_ -#define SRC_ETW_INL_H_ +#ifndef SRC_NODE_WIN32_ETW_PROVIDER_INL_H_ +#define SRC_NODE_WIN32_ETW_PROVIDER_INL_H_ #include "node_win32_etw_provider.h" #include "node_etw_provider.h" namespace node { -using namespace v8; - // From node_win32_etw_provider.cc extern REGHANDLE node_provider; extern EventWriteFunc event_write; @@ -37,7 +35,7 @@ extern int events_enabled; #define ETW_WRITE_STRING_DATA(data_descriptor, data) \ EventDataDescCreate(data_descriptor, \ data, \ - (strlen(data) + 1) * sizeof(char)); + (strlen(data) + 1) * sizeof(*data)); #define ETW_WRITE_INT32_DATA(data_descriptor, data) \ EventDataDescCreate(data_descriptor, data, sizeof(int32_t)); @@ -106,7 +104,8 @@ extern int events_enabled; #define ETW_WRITE_EVENT(eventDescriptor, dataDescriptors) \ DWORD status = event_write(node_provider, \ &eventDescriptor, \ - sizeof(dataDescriptors)/sizeof(*dataDescriptors),\ + sizeof(dataDescriptors) / \ + sizeof(*dataDescriptors), \ dataDescriptors); \ assert(status == ERROR_SUCCESS); @@ -163,7 +162,7 @@ void NODE_NET_STREAM_END(node_dtrace_connection_t* conn, } -void NODE_GC_START(GCType type, GCCallbackFlags flags) { +void NODE_GC_START(v8::GCType type, v8::GCCallbackFlags flags) { if (events_enabled > 0) { EVENT_DATA_DESCRIPTOR descriptors[2]; ETW_WRITE_GC(descriptors, type, flags); @@ -172,7 +171,7 @@ void NODE_GC_START(GCType type, GCCallbackFlags flags) { } -void NODE_GC_DONE(GCType type, GCCallbackFlags flags) { +void NODE_GC_DONE(v8::GCType type, v8::GCCallbackFlags flags) { if (events_enabled > 0) { EVENT_DATA_DESCRIPTOR descriptors[2]; ETW_WRITE_GC(descriptors, type, flags); @@ -221,7 +220,12 @@ void NODE_V8SYMBOL_ADD(LPCSTR symbol, if (symbol == NULL) { SETSYMBUF(L"NULL"); } else { - symbol_len = MultiByteToWideChar(CP_ACP, 0, symbol, symbol_len, symbuf, 128); + symbol_len = MultiByteToWideChar(CP_ACP, + 0, + symbol, + symbol_len, + symbuf, + 128); if (symbol_len == 0) { SETSYMBUF(L"Invalid"); } else { @@ -267,5 +271,7 @@ bool NODE_NET_STREAM_END_ENABLED() { return events_enabled > 0; } bool NODE_NET_SOCKET_READ_ENABLED() { return events_enabled > 0; } bool NODE_NET_SOCKET_WRITE_ENABLED() { return events_enabled > 0; } bool NODE_V8SYMBOL_ENABLED() { return events_enabled > 0; } -} -#endif // SRC_ETW_INL_H_ + +} // namespace node + +#endif // SRC_NODE_WIN32_ETW_PROVIDER_INL_H_ diff --git a/src/node_win32_etw_provider.cc b/src/node_win32_etw_provider.cc index 258005c8b8..73f3e817c0 100644 --- a/src/node_win32_etw_provider.cc +++ b/src/node_win32_etw_provider.cc @@ -26,8 +26,6 @@ namespace node { -using namespace v8; - HMODULE advapi; REGHANDLE node_provider; EventRegisterFunc event_register; @@ -177,7 +175,7 @@ void init_etw() { etw_events_change_async); uv_unref(reinterpret_cast(&dispatch_etw_events_change_async)); - if (event_register) { + if (event_register) { DWORD status = event_register(&NODE_ETW_PROVIDER, etw_events_enable_callback, NULL, diff --git a/src/node_win32_etw_provider.h b/src/node_win32_etw_provider.h index b47b87cf48..d0a41fe5dc 100644 --- a/src/node_win32_etw_provider.h +++ b/src/node_win32_etw_provider.h @@ -19,16 +19,14 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef SRC_ETW_H_ -#define SRC_ETW_H_ +#ifndef SRC_NODE_WIN32_ETW_PROVIDER_H_ +#define SRC_NODE_WIN32_ETW_PROVIDER_H_ #include #include "node_dtrace.h" namespace node { -using namespace v8; - #if defined(_MSC_VER) # define INLINE __forceinline #else @@ -70,8 +68,8 @@ INLINE void NODE_NET_SERVER_CONNECTION(node_dtrace_connection_t* conn, const char *remote, int port, int fd); INLINE void NODE_NET_STREAM_END(node_dtrace_connection_t* conn, const char *remote, int port, int fd); -INLINE void NODE_GC_START(GCType type, GCCallbackFlags flags); -INLINE void NODE_GC_DONE(GCType type, GCCallbackFlags flags); +INLINE void NODE_GC_START(v8::GCType type, v8::GCCallbackFlags flags); +INLINE void NODE_GC_DONE(v8::GCType type, v8::GCCallbackFlags flags); INLINE void NODE_V8SYMBOL_REMOVE(const void* addr1, const void* addr2); INLINE void NODE_V8SYMBOL_MOVE(const void* addr1, const void* addr2); INLINE void NODE_V8SYMBOL_RESET(); @@ -92,5 +90,6 @@ INLINE bool NODE_V8SYMBOL_ENABLED(); #define NODE_NET_SOCKET_READ(arg0, arg1) #define NODE_NET_SOCKET_WRITE(arg0, arg1) -} -#endif // SRC_ETW_H_ +} // namespace node + +#endif // SRC_NODE_WIN32_ETW_PROVIDER_H_ diff --git a/src/node_win32_perfctr_provider.cc b/src/node_win32_perfctr_provider.cc index 903150ed61..db8a9207f1 100644 --- a/src/node_win32_perfctr_provider.cc +++ b/src/node_win32_perfctr_provider.cc @@ -29,72 +29,61 @@ typedef ULONG (WINAPI *PerfStartProviderExFunc)( __in LPGUID ProviderGuid, __in_opt PPERF_PROVIDER_CONTEXT ProviderContext, - __out PHANDLE Provider - ); + __out PHANDLE Provider); typedef ULONG (WINAPI *PerfStopProviderFunc)( - __in HANDLE ProviderHandle - ); + __in HANDLE ProviderHandle); typedef ULONG (WINAPI *PerfSetCounterSetInfoFunc)( __in HANDLE ProviderHandle, __inout_bcount(TemplateSize) PPERF_COUNTERSET_INFO Template, - __in ULONG TemplateSize - ); + __in ULONG TemplateSize); typedef PPERF_COUNTERSET_INSTANCE (WINAPI *PerfCreateInstanceFunc)( __in HANDLE ProviderHandle, __in LPCGUID CounterSetGuid, __in PCWSTR Name, - __in ULONG Id - ); + __in ULONG Id); typedef ULONG (WINAPI *PerfDeleteInstanceFunc)( __in HANDLE Provider, - __in PPERF_COUNTERSET_INSTANCE InstanceBlock - ); + __in PPERF_COUNTERSET_INSTANCE InstanceBlock); typedef ULONG (WINAPI *PerfSetULongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONG Value - ); + __in ULONG Value); typedef ULONG (WINAPI *PerfSetULongLongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONGLONG Value - ); + __in ULONGLONG Value); typedef ULONG (WINAPI *PerfIncrementULongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONG Value - ); + __in ULONG Value); typedef ULONG (WINAPI *PerfIncrementULongLongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONGLONG Value - ); + __in ULONGLONG Value); typedef ULONG (WINAPI *PerfDecrementULongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONG Value - ); + __in ULONG Value); typedef ULONG (WINAPI *PerfDecrementULongLongCounterValueFunc)( __in HANDLE Provider, __inout PPERF_COUNTERSET_INSTANCE Instance, __in ULONG CounterId, - __in ULONGLONG Value - ); + __in ULONGLONG Value); HMODULE advapimod; @@ -341,5 +330,4 @@ void NODE_COUNT_PIPE_BYTES_RECV(int bytes) { } } - -} +} // namespace node diff --git a/src/node_win32_perfctr_provider.h b/src/node_win32_perfctr_provider.h index 1b16a3fec8..d62fb92ced 100644 --- a/src/node_win32_perfctr_provider.h +++ b/src/node_win32_perfctr_provider.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef SRC_WINPERFCTRS_H_ -#define SRC_WINPERFCTRS_H_ +#ifndef SRC_NODE_WIN32_PERFCTR_PROVIDER_H_ +#define SRC_NODE_WIN32_PERFCTR_PROVIDER_H_ #if defined(_MSC_VER) # define INLINE __forceinline @@ -49,7 +49,6 @@ void NODE_COUNT_PIPE_BYTES_RECV(int bytes); void InitPerfCountersWin32(); void TermPerfCountersWin32(); -} - -#endif +} // namespace node +#endif // SRC_NODE_WIN32_PERFCTR_PROVIDER_H_ diff --git a/src/node_wrap.h b/src/node_wrap.h index ff7179fdb8..c03b30eb4d 100644 --- a/src/node_wrap.h +++ b/src/node_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_WRAP_H -#define NODE_WRAP_H +#ifndef SRC_NODE_WRAP_H_ +#define SRC_NODE_WRAP_H_ #include "v8.h" #include "uv.h" @@ -63,6 +63,6 @@ inline uv_stream_t* HandleToStream(v8::Local obj) { return NULL; } -} +} // namespace node -#endif +#endif // SRC_NODE_WRAP_H_ diff --git a/src/node_zlib.cc b/src/node_zlib.cc index 07f44d126e..8504d73fbc 100644 --- a/src/node_zlib.cc +++ b/src/node_zlib.cc @@ -68,21 +68,19 @@ void InitZlib(v8::Handle target); class ZCtx : public ObjectWrap { public: - ZCtx(node_zlib_mode mode) - : ObjectWrap() - , init_done_(false) - , level_(0) - , windowBits_(0) - , memLevel_(0) - , strategy_(0) - , err_(0) - , dictionary_(NULL) - , dictionary_len_(0) - , flush_(0) - , chunk_size_(0) - , write_in_progress_(false) - , mode_(mode) - { + explicit ZCtx(node_zlib_mode mode) : ObjectWrap(), + init_done_(false), + level_(0), + windowBits_(0), + memLevel_(0), + strategy_(0), + err_(0), + dictionary_(NULL), + dictionary_len_(0), + flush_(0), + chunk_size_(0), + write_in_progress_(false), + mode_(mode) { } @@ -219,17 +217,14 @@ class ZCtx : public ObjectWrap { // If data was encoded with dictionary if (ctx->err_ == Z_NEED_DICT && ctx->dictionary_ != NULL) { - // Load it ctx->err_ = inflateSetDictionary(&ctx->strm_, ctx->dictionary_, ctx->dictionary_len_); if (ctx->err_ == Z_OK) { - // And try to decode again ctx->err_ = inflate(&ctx->strm_, ctx->flush_); } else if (ctx->err_ == Z_DATA_ERROR) { - // Both inflateSetDictionary() and inflate() return Z_DATA_ERROR. // Make it possible for After() to tell a bad dictionary from bad // input. @@ -510,8 +505,8 @@ class ZCtx : public ObjectWrap { } private: - static const int kDeflateContextSize = 16384; // approximate - static const int kInflateContextSize = 10240; // approximate + static const int kDeflateContextSize = 16384; // approximate + static const int kInflateContextSize = 10240; // approximate bool init_done_; diff --git a/src/pipe_wrap.cc b/src/pipe_wrap.cc index 1544789490..7a7b93e051 100644 --- a/src/pipe_wrap.cc +++ b/src/pipe_wrap.cc @@ -49,7 +49,7 @@ static Cached onconnection_sym; static Cached oncomplete_sym; -// TODO share with TCPWrap? +// TODO(bnoordhuis) share with TCPWrap? typedef class ReqWrap ConnectWrap; @@ -100,7 +100,9 @@ void PipeWrap::Initialize(Handle target) { NODE_SET_PROTOTYPE_METHOD(t, "shutdown", StreamWrap::Shutdown); NODE_SET_PROTOTYPE_METHOD(t, "writeBuffer", StreamWrap::WriteBuffer); - NODE_SET_PROTOTYPE_METHOD(t, "writeAsciiString", StreamWrap::WriteAsciiString); + NODE_SET_PROTOTYPE_METHOD(t, + "writeAsciiString", + StreamWrap::WriteAsciiString); NODE_SET_PROTOTYPE_METHOD(t, "writeUtf8String", StreamWrap::WriteUtf8String); NODE_SET_PROTOTYPE_METHOD(t, "writeUcs2String", StreamWrap::WriteUcs2String); @@ -134,8 +136,8 @@ void PipeWrap::New(const FunctionCallbackInfo& args) { PipeWrap::PipeWrap(Handle object, bool ipc) : StreamWrap(object, reinterpret_cast(&handle_)) { int r = uv_pipe_init(uv_default_loop(), &handle_, ipc); - assert(r == 0); // How do we proxy this error up to javascript? - // Suggestion: uv_pipe_init() returns void. + assert(r == 0); // How do we proxy this error up to javascript? + // Suggestion: uv_pipe_init() returns void. UpdateWriteQueueSize(); } @@ -177,7 +179,7 @@ void PipeWrap::Listen(const FunctionCallbackInfo& args) { } -// TODO maybe share with TCPWrap? +// TODO(bnoordhuis) maybe share with TCPWrap? void PipeWrap::OnConnection(uv_stream_t* handle, int status) { HandleScope scope(node_isolate); @@ -217,10 +219,10 @@ void PipeWrap::OnConnection(uv_stream_t* handle, int status) { MakeCallback(wrap->object(), onconnection_sym, ARRAY_SIZE(argv), argv); } -// TODO Maybe share this with TCPWrap? +// TODO(bnoordhuis) Maybe share this with TCPWrap? void PipeWrap::AfterConnect(uv_connect_t* req, int status) { - ConnectWrap* req_wrap = (ConnectWrap*) req->data; - PipeWrap* wrap = (PipeWrap*) req->handle->data; + ConnectWrap* req_wrap = reinterpret_cast(req->data); + PipeWrap* wrap = reinterpret_cast(req->handle->data); HandleScope scope(node_isolate); diff --git a/src/pipe_wrap.h b/src/pipe_wrap.h index ec6be31733..944835774d 100644 --- a/src/pipe_wrap.h +++ b/src/pipe_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef PIPE_WRAP_H_ -#define PIPE_WRAP_H_ +#ifndef SRC_PIPE_WRAP_H_ +#define SRC_PIPE_WRAP_H_ #include "stream_wrap.h" namespace node { @@ -57,4 +57,4 @@ class PipeWrap : public StreamWrap { } // namespace node -#endif // PIPE_WRAP_H_ +#endif // SRC_PIPE_WRAP_H_ diff --git a/src/process_wrap.cc b/src/process_wrap.cc index 5044c124e8..76376366e3 100644 --- a/src/process_wrap.cc +++ b/src/process_wrap.cc @@ -75,7 +75,7 @@ class ProcessWrap : public HandleWrap { new ProcessWrap(args.This()); } - ProcessWrap(Handle object) + explicit ProcessWrap(Handle object) : HandleWrap(object, reinterpret_cast(&process_)) { } @@ -158,7 +158,7 @@ class ProcessWrap : public HandleWrap { return ThrowTypeError("options.gid should be a number"); } - // TODO is this possible to do without mallocing ? + // TODO(bnoordhuis) is this possible to do without mallocing ? // options.file Local file_v = js_options->Get(String::NewSymbol("file")); @@ -195,7 +195,7 @@ class ProcessWrap : public HandleWrap { if (!env_v.IsEmpty() && env_v->IsArray()) { Local env = Local::Cast(env_v); int envc = env->Length(); - options.env = new char*[envc + 1]; // Heap allocated to detect errors. + options.env = new char*[envc + 1]; // Heap allocated to detect errors. for (int i = 0; i < envc; i++) { String::Utf8Value pair(env->Get(i)); options.env[i] = strdup(*pair); @@ -212,7 +212,7 @@ class ProcessWrap : public HandleWrap { options.flags |= UV_PROCESS_WINDOWS_VERBATIM_ARGUMENTS; } - //options.detached + // options.detached if (js_options->Get(String::NewSymbol("detached"))->IsTrue()) { options.flags |= UV_PROCESS_DETACHED; } diff --git a/src/queue.h b/src/queue.h index 0a3783b9f1..fa42e7f1cf 100644 --- a/src/queue.h +++ b/src/queue.h @@ -13,8 +13,8 @@ * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. */ -#ifndef QUEUE_H_ -#define QUEUE_H_ +#ifndef SRC_QUEUE_H_ +#define SRC_QUEUE_H_ typedef void *QUEUE[2]; @@ -89,4 +89,4 @@ typedef void *QUEUE[2]; } \ while (0) -#endif /* QUEUE_H_ */ +#endif /* SRC_QUEUE_H_ */ diff --git a/src/req_wrap.h b/src/req_wrap.h index 6dbf080a43..d4c1180ded 100644 --- a/src/req_wrap.h +++ b/src/req_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef REQ_WRAP_H_ -#define REQ_WRAP_H_ +#ifndef SRC_REQ_WRAP_H_ +#define SRC_REQ_WRAP_H_ #include "queue.h" @@ -33,7 +33,7 @@ extern QUEUE req_wrap_queue; template class ReqWrap { -public: + public: ReqWrap(v8::Handle object = v8::Handle()) { v8::HandleScope scope(node_isolate); if (object.IsEmpty()) object = v8::Object::New(); @@ -79,11 +79,11 @@ public: v8::Persistent object_; QUEUE req_wrap_queue_; void* data_; - T req_; // *must* be last, GetActiveRequests() in node.cc depends on it + T req_; // *must* be last, GetActiveRequests() in node.cc depends on it }; } // namespace node -#endif // REQ_WRAP_H_ +#endif // SRC_REQ_WRAP_H_ diff --git a/src/signal_wrap.cc b/src/signal_wrap.cc index 2120a2f866..3f5df4e90a 100644 --- a/src/signal_wrap.cc +++ b/src/signal_wrap.cc @@ -72,7 +72,7 @@ class SignalWrap : public HandleWrap { new SignalWrap(args.This()); } - SignalWrap(Handle object) + explicit SignalWrap(Handle object) : HandleWrap(object, reinterpret_cast(&handle_)) { int r = uv_signal_init(uv_default_loop(), &handle_); assert(r == 0); diff --git a/src/smalloc.h b/src/smalloc.h index be4515b487..47ab4f2d8c 100644 --- a/src/smalloc.h +++ b/src/smalloc.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef NODE_SMALLOC_H_ -#define NODE_SMALLOC_H_ +#ifndef SRC_SMALLOC_H_ +#define SRC_SMALLOC_H_ #include "node.h" #include "v8.h" @@ -64,7 +64,6 @@ NODE_EXTERN void Alloc(v8::Handle obj, NODE_EXTERN void AllocDispose(v8::Handle obj); } // namespace smalloc - } // namespace node -#endif // NODE_SMALLOC_H_ +#endif // SRC_SMALLOC_H_ diff --git a/src/stream_wrap.cc b/src/stream_wrap.cc index e52dcf7d8b..d5734d527d 100644 --- a/src/stream_wrap.cc +++ b/src/stream_wrap.cc @@ -29,8 +29,8 @@ #include "udp_wrap.h" #include "node_counters.h" -#include // abort() -#include // INT_MAX +#include // abort() +#include // INT_MAX namespace node { @@ -217,7 +217,7 @@ void StreamWrap::WriteBuffer(const FunctionCallbackInfo& args) { size_t length = Buffer::Length(buf_obj); char* storage = new char[sizeof(WriteWrap)]; - WriteWrap* req_wrap = new (storage) WriteWrap(req_wrap_obj, wrap); + WriteWrap* req_wrap = new(storage) WriteWrap(req_wrap_obj, wrap); req_wrap_obj->SetHiddenValue(buffer_sym, buf_obj); @@ -269,7 +269,7 @@ void StreamWrap::WriteStringImpl(const FunctionCallbackInfo& args) { } char* storage = new char[sizeof(WriteWrap) + storage_size + 15]; - WriteWrap* req_wrap = new (storage) WriteWrap(req_wrap_obj, wrap); + WriteWrap* req_wrap = new(storage) WriteWrap(req_wrap_obj, wrap); char* data = reinterpret_cast(ROUND_UP( reinterpret_cast(storage) + sizeof(WriteWrap), 16)); @@ -377,7 +377,7 @@ void StreamWrap::Writev(const FunctionCallbackInfo& args) { storage_size += sizeof(WriteWrap); char* storage = new char[storage_size]; - WriteWrap* req_wrap = new (storage) WriteWrap(req_wrap_obj, wrap); + WriteWrap* req_wrap = new(storage) WriteWrap(req_wrap_obj, wrap); uint32_t bytes = 0; size_t offset = sizeof(WriteWrap); @@ -616,4 +616,4 @@ Handle StreamWrapCallbacks::Self() { return wrap_->object(); } -} +} // namespace node diff --git a/src/stream_wrap.h b/src/stream_wrap.h index 0a431c1486..98d22cbfad 100644 --- a/src/stream_wrap.h +++ b/src/stream_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef STREAM_WRAP_H_ -#define STREAM_WRAP_H_ +#ifndef SRC_STREAM_WRAP_H_ +#define SRC_STREAM_WRAP_H_ #include "v8.h" #include "node.h" @@ -53,8 +53,8 @@ class WriteWrap: public ReqWrap { protected: // People should not be using the non-placement new and delete operator on a // WriteWrap. Ensure this never happens. - void* operator new(size_t size) { assert(0); }; - void operator delete(void* ptr) { assert(0); }; + void* operator new(size_t size) { assert(0); } + void operator delete(void* ptr) { assert(0); } }; // Overridable callbacks' types @@ -160,4 +160,4 @@ class StreamWrap : public HandleWrap { } // namespace node -#endif // STREAM_WRAP_H_ +#endif // SRC_STREAM_WRAP_H_ diff --git a/src/string_bytes.cc b/src/string_bytes.cc index c616c179c4..75a7173dfd 100644 --- a/src/string_bytes.cc +++ b/src/string_bytes.cc @@ -458,7 +458,7 @@ static bool contains_non_ascii(const char* src, size_t len) { return contains_non_ascii_slow(src, len); } - const unsigned bytes_per_word = sizeof(void*); + const unsigned bytes_per_word = sizeof(uintptr_t); const unsigned align_mask = bytes_per_word - 1; const unsigned unaligned = reinterpret_cast(src) & align_mask; @@ -505,7 +505,7 @@ static void force_ascii(const char* src, char* dst, size_t len) { return; } - const unsigned bytes_per_word = sizeof(void*); + const unsigned bytes_per_word = sizeof(uintptr_t); const unsigned align_mask = bytes_per_word - 1; const unsigned src_unalign = reinterpret_cast(src) & align_mask; const unsigned dst_unalign = reinterpret_cast(dst) & align_mask; diff --git a/src/tcp_wrap.cc b/src/tcp_wrap.cc index 9be4121063..010e9c3ec5 100644 --- a/src/tcp_wrap.cc +++ b/src/tcp_wrap.cc @@ -96,7 +96,9 @@ void TCPWrap::Initialize(Handle target) { NODE_SET_PROTOTYPE_METHOD(t, "shutdown", StreamWrap::Shutdown); NODE_SET_PROTOTYPE_METHOD(t, "writeBuffer", StreamWrap::WriteBuffer); - NODE_SET_PROTOTYPE_METHOD(t, "writeAsciiString", StreamWrap::WriteAsciiString); + NODE_SET_PROTOTYPE_METHOD(t, + "writeAsciiString", + StreamWrap::WriteAsciiString); NODE_SET_PROTOTYPE_METHOD(t, "writeUtf8String", StreamWrap::WriteUtf8String); NODE_SET_PROTOTYPE_METHOD(t, "writeUcs2String", StreamWrap::WriteUcs2String); NODE_SET_PROTOTYPE_METHOD(t, "writev", StreamWrap::Writev); @@ -113,7 +115,9 @@ void TCPWrap::Initialize(Handle target) { NODE_SET_PROTOTYPE_METHOD(t, "setKeepAlive", SetKeepAlive); #ifdef _WIN32 - NODE_SET_PROTOTYPE_METHOD(t, "setSimultaneousAccepts", SetSimultaneousAccepts); + NODE_SET_PROTOTYPE_METHOD(t, + "setSimultaneousAccepts", + SetSimultaneousAccepts); #endif onconnection_sym = String::New("onconnection"); @@ -151,8 +155,8 @@ void TCPWrap::New(const FunctionCallbackInfo& args) { TCPWrap::TCPWrap(Handle object) : StreamWrap(object, reinterpret_cast(&handle_)) { int r = uv_tcp_init(uv_default_loop(), &handle_); - assert(r == 0); // How do we proxy this error up to javascript? - // Suggestion: uv_tcp_init() returns void. + assert(r == 0); // How do we proxy this error up to javascript? + // Suggestion: uv_tcp_init() returns void. UpdateWriteQueueSize(); } @@ -331,8 +335,8 @@ void TCPWrap::OnConnection(uv_stream_t* handle, int status) { void TCPWrap::AfterConnect(uv_connect_t* req, int status) { - ConnectWrap* req_wrap = (ConnectWrap*) req->data; - TCPWrap* wrap = (TCPWrap*) req->handle->data; + ConnectWrap* req_wrap = reinterpret_cast(req->data); + TCPWrap* wrap = reinterpret_cast(req->handle->data); HandleScope scope(node_isolate); diff --git a/src/tcp_wrap.h b/src/tcp_wrap.h index 317557d4a7..374458b78d 100644 --- a/src/tcp_wrap.h +++ b/src/tcp_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef TCP_WRAP_H_ -#define TCP_WRAP_H_ +#ifndef SRC_TCP_WRAP_H_ +#define SRC_TCP_WRAP_H_ #include "stream_wrap.h" namespace node { @@ -34,7 +34,7 @@ class TCPWrap : public StreamWrap { uv_tcp_t* UVHandle(); private: - TCPWrap(v8::Handle object); + explicit TCPWrap(v8::Handle object); ~TCPWrap(); static void New(const v8::FunctionCallbackInfo& args); @@ -64,4 +64,4 @@ class TCPWrap : public StreamWrap { } // namespace node -#endif // TCP_WRAP_H_ +#endif // SRC_TCP_WRAP_H_ diff --git a/src/timer_wrap.cc b/src/timer_wrap.cc index 1a3d43cc05..8ca3958677 100644 --- a/src/timer_wrap.cc +++ b/src/timer_wrap.cc @@ -75,7 +75,7 @@ class TimerWrap : public HandleWrap { new TimerWrap(args.This()); } - TimerWrap(Handle object) + explicit TimerWrap(Handle object) : HandleWrap(object, reinterpret_cast(&handle_)) { int r = uv_timer_init(uv_default_loop(), &handle_); assert(r == 0); diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc index 7a4220a7dc..b5c775aa17 100644 --- a/src/tls_wrap.cc +++ b/src/tls_wrap.cc @@ -442,29 +442,29 @@ Handle TLSCallbacks::GetSSLError(int status, int* err) { *err = SSL_get_error(ssl_, status); switch (*err) { - case SSL_ERROR_NONE: - case SSL_ERROR_WANT_READ: - case SSL_ERROR_WANT_WRITE: - break; - case SSL_ERROR_ZERO_RETURN: - return scope.Close(String::NewSymbol("ZERO_RETURN")); - break; - default: - { - BUF_MEM* mem; - BIO* bio; + case SSL_ERROR_NONE: + case SSL_ERROR_WANT_READ: + case SSL_ERROR_WANT_WRITE: + break; + case SSL_ERROR_ZERO_RETURN: + return scope.Close(String::NewSymbol("ZERO_RETURN")); + break; + default: + { + BUF_MEM* mem; + BIO* bio; - assert(*err == SSL_ERROR_SSL || *err == SSL_ERROR_SYSCALL); + assert(*err == SSL_ERROR_SSL || *err == SSL_ERROR_SYSCALL); - bio = BIO_new(BIO_s_mem()); - assert(bio != NULL); - ERR_print_errors(bio); - BIO_get_mem_ptr(bio, &mem); - Handle r = Exception::Error(String::New(mem->data, mem->length)); - BIO_free_all(bio); + bio = BIO_new(BIO_s_mem()); + assert(bio != NULL); + ERR_print_errors(bio); + BIO_get_mem_ptr(bio, &mem); + Handle r = Exception::Error(String::New(mem->data, mem->length)); + BIO_free_all(bio); - return scope.Close(r); - } + return scope.Close(r); + } } return Handle(); } @@ -686,104 +686,104 @@ void TLSCallbacks::ParseClientHello() { Handle argv[1]; switch (hello_.state) { - case kParseWaiting: - // >= 5 bytes for header parsing - if (avail < 5) - break; - - if (data[0] == kChangeCipherSpec || - data[0] == kAlert || - data[0] == kHandshake || - data[0] == kApplicationData) { - hello_.frame_len = (data[3] << 8) + data[4]; - hello_.state = kParseTLSHeader; - hello_.body_offset = 5; - } else { - hello_.frame_len = (data[0] << 8) + data[1]; - hello_.state = kParseSSLHeader; - if (*data & 0x40) { - // header with padding - hello_.body_offset = 3; + case kParseWaiting: + // >= 5 bytes for header parsing + if (avail < 5) + break; + + if (data[0] == kChangeCipherSpec || + data[0] == kAlert || + data[0] == kHandshake || + data[0] == kApplicationData) { + hello_.frame_len = (data[3] << 8) + data[4]; + hello_.state = kParseTLSHeader; + hello_.body_offset = 5; } else { - // without padding - hello_.body_offset = 2; + hello_.frame_len = (data[0] << 8) + data[1]; + hello_.state = kParseSSLHeader; + if (*data & 0x40) { + // header with padding + hello_.body_offset = 3; + } else { + // without padding + hello_.body_offset = 2; + } } - } - - // Sanity check (too big frame, or too small) - // Let OpenSSL handle it - if (hello_.frame_len >= kMaxTLSFrameLen) - return ParseFinish(); - // Fall through - case kParseTLSHeader: - case kParseSSLHeader: - // >= 5 + frame size bytes for frame parsing - if (avail < hello_.body_offset + hello_.frame_len) - break; + // Sanity check (too big frame, or too small) + // Let OpenSSL handle it + if (hello_.frame_len >= kMaxTLSFrameLen) + return ParseFinish(); - // Skip unsupported frames and gather some data from frame + // Fall through + case kParseTLSHeader: + case kParseSSLHeader: + // >= 5 + frame size bytes for frame parsing + if (avail < hello_.body_offset + hello_.frame_len) + break; + + // Skip unsupported frames and gather some data from frame + + // TODO(indutny): Check protocol version + if (data[hello_.body_offset] == kClientHello) { + is_clienthello = true; + uint8_t* body; + size_t session_offset; + + if (hello_.state == kParseTLSHeader) { + // Skip frame header, hello header, protocol version and random data + session_offset = hello_.body_offset + 4 + 2 + 32; + + if (session_offset + 1 < avail) { + body = data + session_offset; + session_size = *body; + session_id = body + 1; + } + } else if (hello_.state == kParseSSLHeader) { + // Skip header, version + session_offset = hello_.body_offset + 3; - // TODO(indutny): Check protocol version - if (data[hello_.body_offset] == kClientHello) { - is_clienthello = true; - uint8_t* body; - size_t session_offset; + if (session_offset + 4 < avail) { + body = data + session_offset; - if (hello_.state == kParseTLSHeader) { - // Skip frame header, hello header, protocol version and random data - session_offset = hello_.body_offset + 4 + 2 + 32; + int ciphers_size = (body[0] << 8) + body[1]; - if (session_offset + 1 < avail) { - body = data + session_offset; - session_size = *body; - session_id = body + 1; + if (body + 4 + ciphers_size < data + avail) { + session_size = (body[2] << 8) + body[3]; + session_id = body + 4 + ciphers_size; + } + } + } else { + // Whoa? How did we get here? + abort(); } - } else if (hello_.state == kParseSSLHeader) { - // Skip header, version - session_offset = hello_.body_offset + 3; - - if (session_offset + 4 < avail) { - body = data + session_offset; - - int ciphers_size = (body[0] << 8) + body[1]; - if (body + 4 + ciphers_size < data + avail) { - session_size = (body[2] << 8) + body[3]; - session_id = body + 4 + ciphers_size; - } + // Check if we overflowed (do not reply with any private data) + if (session_id == NULL || + session_size > 32 || + session_id + session_size > data + avail) { + return ParseFinish(); } - } else { - // Whoa? How did we get here? - abort(); - } - // Check if we overflowed (do not reply with any private data) - if (session_id == NULL || - session_size > 32 || - session_id + session_size > data + avail) { - return ParseFinish(); + // TODO(indutny): Parse other things? } - // TODO(indutny): Parse other things? - } - - // Not client hello - let OpenSSL handle it - if (!is_clienthello) - return ParseFinish(); + // Not client hello - let OpenSSL handle it + if (!is_clienthello) + return ParseFinish(); - hello_.state = kParsePaused; - hello_obj = Object::New(); - hello_obj->Set(sessionid_sym, - Buffer::New(reinterpret_cast(session_id), - session_size)); + hello_.state = kParsePaused; + hello_obj = Object::New(); + hello_obj->Set(sessionid_sym, + Buffer::New(reinterpret_cast(session_id), + session_size)); - argv[0] = hello_obj; - MakeCallback(object(), onclienthello_sym, 1, argv); - break; - case kParseEnded: - default: - break; + argv[0] = hello_obj; + MakeCallback(object(), onclienthello_sym, 1, argv); + break; + case kParseEnded: + default: + break; } } @@ -1183,17 +1183,17 @@ int TLSCallbacks::SelectNextProtoCallback(SSL* s, int status = SSL_select_next_proto(out, outlen, in, inlen, npn_protos, len); Handle result; switch (status) { - case OPENSSL_NPN_UNSUPPORTED: - result = Null(node_isolate); - break; - case OPENSSL_NPN_NEGOTIATED: - result = String::New(reinterpret_cast(*out), *outlen); - break; - case OPENSSL_NPN_NO_OVERLAP: - result = False(node_isolate); - break; - default: - break; + case OPENSSL_NPN_UNSUPPORTED: + result = Null(node_isolate); + break; + case OPENSSL_NPN_NEGOTIATED: + result = String::New(reinterpret_cast(*out), *outlen); + break; + case OPENSSL_NPN_NO_OVERLAP: + result = False(node_isolate); + break; + default: + break; } if (!result.IsEmpty()) diff --git a/src/tty_wrap.cc b/src/tty_wrap.cc index df34e56b13..d3c2a348d6 100644 --- a/src/tty_wrap.cc +++ b/src/tty_wrap.cc @@ -69,7 +69,9 @@ void TTYWrap::Initialize(Handle target) { NODE_SET_PROTOTYPE_METHOD(t, "readStop", StreamWrap::ReadStop); NODE_SET_PROTOTYPE_METHOD(t, "writeBuffer", StreamWrap::WriteBuffer); - NODE_SET_PROTOTYPE_METHOD(t, "writeAsciiString", StreamWrap::WriteAsciiString); + NODE_SET_PROTOTYPE_METHOD(t, + "writeAsciiString", + StreamWrap::WriteAsciiString); NODE_SET_PROTOTYPE_METHOD(t, "writeUtf8String", StreamWrap::WriteUtf8String); NODE_SET_PROTOTYPE_METHOD(t, "writeUcs2String", StreamWrap::WriteUcs2String); diff --git a/src/tty_wrap.h b/src/tty_wrap.h index 898bd11573..0475073fdc 100644 --- a/src/tty_wrap.h +++ b/src/tty_wrap.h @@ -19,8 +19,8 @@ // OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE // USE OR OTHER DEALINGS IN THE SOFTWARE. -#ifndef TTY_WRAP_H_ -#define TTY_WRAP_H_ +#ifndef SRC_TTY_WRAP_H_ +#define SRC_TTY_WRAP_H_ #include "handle_wrap.h" #include "stream_wrap.h" @@ -46,6 +46,6 @@ class TTYWrap : public StreamWrap { uv_tty_t handle_; }; -} // namespace node +} // namespace node -#endif // TTY_WRAP_H_ +#endif // SRC_TTY_WRAP_H_ diff --git a/src/udp_wrap.cc b/src/udp_wrap.cc index 7b335ec2e0..a59843fb6d 100644 --- a/src/udp_wrap.cc +++ b/src/udp_wrap.cc @@ -57,7 +57,7 @@ static Cached onmessage_sym; UDPWrap::UDPWrap(Handle object) : HandleWrap(object, reinterpret_cast(&handle_)) { int r = uv_udp_init(uv_default_loop(), &handle_); - assert(r == 0); // can't fail anyway + assert(r == 0); // can't fail anyway } @@ -328,7 +328,7 @@ void UDPWrap::GetSockName(const FunctionCallbackInfo& args) { } -// TODO share with StreamWrap::AfterWrite() in stream_wrap.cc +// TODO(bnoordhuis) share with StreamWrap::AfterWrite() in stream_wrap.cc void UDPWrap::OnSend(uv_udp_send_t* req, int status) { HandleScope scope(node_isolate); @@ -413,6 +413,6 @@ uv_udp_t* UDPWrap::UVHandle() { } -} // namespace node +} // namespace node NODE_MODULE(node_udp_wrap, node::UDPWrap::Initialize) diff --git a/src/udp_wrap.h b/src/udp_wrap.h index 878ca2046c..fb73647045 100644 --- a/src/udp_wrap.h +++ b/src/udp_wrap.h @@ -1,5 +1,26 @@ -#ifndef UDP_WRAP_H_ -#define UDP_WRAP_H_ +// Copyright Joyent, Inc. and other Node contributors. +// +// Permission is hereby granted, free of charge, to any person obtaining a +// copy of this software and associated documentation files (the +// "Software"), to deal in the Software without restriction, including +// without limitation the rights to use, copy, modify, merge, publish, +// distribute, sublicense, and/or sell copies of the Software, and to permit +// persons to whom the Software is furnished to do so, subject to the +// following conditions: +// +// The above copyright notice and this permission notice shall be included +// in all copies or substantial portions of the Software. +// +// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +// USE OR OTHER DEALINGS IN THE SOFTWARE. + +#ifndef SRC_UDP_WRAP_H_ +#define SRC_UDP_WRAP_H_ #include "node.h" #include "req_wrap.h" @@ -33,7 +54,7 @@ class UDPWrap: public HandleWrap { uv_udp_t* UVHandle(); private: - UDPWrap(v8::Handle object); + explicit UDPWrap(v8::Handle object); virtual ~UDPWrap(); static void DoBind(const v8::FunctionCallbackInfo& args, @@ -54,6 +75,6 @@ class UDPWrap: public HandleWrap { uv_udp_t handle_; }; -} // namespace node +} // namespace node -#endif // UDP_WRAP_H_ +#endif // SRC_UDP_WRAP_H_ diff --git a/src/v8abbr.h b/src/v8abbr.h index 95ddb6a661..7a5f9e7fa8 100644 --- a/src/v8abbr.h +++ b/src/v8abbr.h @@ -1,10 +1,32 @@ +/** Copyright Joyent, Inc. and other Node contributors. +* +* Permission is hereby granted, free of charge, to any person obtaining a +* copy of this software and associated documentation files (the +* "Software"), to deal in the Software without restriction, including +* without limitation the rights to use, copy, modify, merge, publish, +* distribute, sublicense, and/or sell copies of the Software, and to permit +* persons to whom the Software is furnished to do so, subject to the +* following conditions: +* +* The above copyright notice and this permission notice shall be included +* in all copies or substantial portions of the Software. +* +* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS +* OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF +* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN +* NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, +* DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR +* OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE +* USE OR OTHER DEALINGS IN THE SOFTWARE. +*/ + /* * This header defines short names for V8 constants for use by the ustack * helper. */ -#ifndef V8_ABBR_H -#define V8_ABBR_H +#ifndef SRC_V8ABBR_H_ +#define SRC_V8ABBR_H_ /* Frame pointer offsets */ #define V8_OFF_FP_FUNC V8DBG_OFF_FP_FUNCTION @@ -83,4 +105,4 @@ #define V8_OFF_MAP_ATTRS \ V8_OFF_HEAP(V8DBG_CLASS_MAP__INSTANCE_ATTRIBUTES__INT) -#endif /* V8_ABBR_H */ +#endif /* SRC_V8ABBR_H_ */ -- cgit v1.2.3