TheCloud/panda3d
1
0
Fork 0
mirror of https://github.com/panda3d/panda3d.git synced 2025-10-04 10:54:24 -04:00
Code Issues Packages Projects Releases Wiki Activity

plugin - read_log - report error out and early out conditions during calls to read_log

Browse Source
This commit is contained in:
Ken Patel 2011-02-12 01:03:00 +00:00
parent 003483f294
commit ae627fb19b
1 changed files with 24 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
direct/src/plugin/p3dMainObject.cxx
View File

@ -437,6 +437,7 @@ call_read_game_log(P3D_object *params[], int num_params) {
} }
// No log available for us. // No log available for us.
nout << "read_game_log: error: game log name unknown" << "\n";
P3DInstanceManager *inst_mgr = P3DInstanceManager::get_global_ptr(); P3DInstanceManager *inst_mgr = P3DInstanceManager::get_global_ptr();
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
@ -467,6 +468,7 @@ call_read_log(P3D_object *params[], int num_params) {
P3DInstanceManager *inst_mgr = P3DInstanceManager::get_global_ptr(); P3DInstanceManager *inst_mgr = P3DInstanceManager::get_global_ptr();
if (num_params < 1) { if (num_params < 1) {
nout << "read_log: error: not enough parameters" << "\n";
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
@ -478,24 +480,28 @@ call_read_log(P3D_object *params[], int num_params) {
if (log_filename.size() < 4 || log_filename.substr(log_filename.size() - 4) != string(".log")) { if (log_filename.size() < 4 || log_filename.substr(log_filename.size() - 4) != string(".log")) {
// Wrong filename extension. // Wrong filename extension.
nout << "read_log: error: invalid filename" << "\n";
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
size_t slash = log_filename.find('/'); size_t slash = log_filename.find('/');
if (slash != string::npos) { if (slash != string::npos) {
// No slashes allowed. // No slashes allowed.
nout << "read_log: error: invalid filename" << "\n";
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
slash = log_filename.find('\\'); slash = log_filename.find('\\');
if (slash != string::npos) { if (slash != string::npos) {
// Nor backslashes. // Nor backslashes.
nout << "read_log: error: invalid filename" << "\n";
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
size_t colon = log_filename.find(':'); size_t colon = log_filename.find(':');
if (colon != string::npos) { if (colon != string::npos) {
// Nor colons, for that matter. // Nor colons, for that matter.
nout << "read_log: error: invalid filename" << "\n";
return inst_mgr->new_undefined_object(); return inst_mgr->new_undefined_object();
} }
@ -604,9 +610,16 @@ read_log_file(const string &log_pathname,
} }
#endif // _WIN32 #endif // _WIN32
// Render log file header to log_data
log_data << "=======================================";
log_data << "=======================================" << "\n";
log_data << "== PandaLog-" << log_pathname << "\n";
// load file // load file
ifstream log(log_pathname.c_str(), ios::in); ifstream log(log_pathname.c_str(), ios::in);
if (!log) { if (!log) {
log_data << "== PandaLog-" << "Error opening file";
log_data << " " << "(" << log_leafname << ")" << "\n";
return; return;
} }
@ -617,6 +630,13 @@ read_log_file(const string &log_pathname,
<< " bytes, tail_bytes = " << tail_bytes << ", head_bytes = " << " bytes, tail_bytes = " << tail_bytes << ", head_bytes = "
<< head_bytes << "\n"; << head_bytes << "\n";
// Early out if the file is empty
if (file_size == (size_t)0) {
log_data << "== PandaLog-" << "Empty File";
log_data << " " << "(" << log_leafname << ")" << "\n";
return;
}
// Check if we are getting the full file // Check if we are getting the full file
size_t full_bytes = 0; size_t full_bytes = 0;
if (file_size <= head_bytes + tail_bytes) { if (file_size <= head_bytes + tail_bytes) {
@ -628,17 +648,14 @@ read_log_file(const string &log_pathname,
// Allocate a temp buffer to hold file data // Allocate a temp buffer to hold file data
size_t buffer_bytes = max(max(full_bytes, head_bytes), tail_bytes) + 1; size_t buffer_bytes = max(max(full_bytes, head_bytes), tail_bytes) + 1;
nout << "allocating " << buffer_bytes << " bytes to read from file.\n"; nout << "allocating " << buffer_bytes << " bytes to read at a time from file of size " << file_size << ".\n";
char *buffer = new char[buffer_bytes]; char *buffer = new char[buffer_bytes];
if (buffer == NULL) { if (buffer == NULL) {
log_data << "== PandaLog-" << "Error allocating buffer";
log_data << " " << "(" << log_leafname << ")" << "\n";
return; return;
} }
// Render log file header to log_data
log_data << "=======================================";
log_data << "=======================================" << "\n";
log_data << "== PandaLog-" << log_pathname << "\n";
// Render log data if full file is to be fetched // Render log data if full file is to be fetched
if (full_bytes > 0) { if (full_bytes > 0) {
log.seekg(0, ios::beg); log.seekg(0, ios::beg);