1
0
Fork 0
mirror of git://git.code.sf.net/p/cdesktopenv/code synced 2025-02-15 04:32:24 +00:00

dtinfo: Resolve coverity warnings related to uninitialised members of classes

This commit is contained in:
Peter Howkins 2018-04-19 23:25:47 +01:00
parent 02d43dabe6
commit 6e7e9d91e1
48 changed files with 167 additions and 32 deletions

View file

@ -47,7 +47,7 @@ static ostrstream& terminate(ostrstream& ost)
#endif #endif
DocParser::DocParser(Resolver &r) DocParser::DocParser(Resolver &r)
: f_resolver(r), : f_resolver(r), f_ignoring_element(0),
#if defined(SC3) || defined(__osf__) #if defined(SC3) || defined(__osf__)
f_buffer(new char[DATA_BUF_SIZ]), f_buffer(new char[DATA_BUF_SIZ]),
f_output(f_buffer, DATA_BUF_SIZ) f_output(f_buffer, DATA_BUF_SIZ)

View file

@ -122,6 +122,10 @@ debug(cerr, base_ds);
end_try; end_try;
} }
#ifdef C_API
f_index_id = 0;
#endif
MESSAGE(cerr, form("info base %s in %s available.", base_name, base_dir)); MESSAGE(cerr, form("info base %s in %s available.", base_name, base_dir));
} }

View file

@ -63,6 +63,8 @@ btree::btree(const char* store_name)
btree_info.prefix = NULL; btree_info.prefix = NULL;
btree_info.lorder = 0; btree_info.lorder = 0;
key_DBT.data = 0;
key_DBT.size = 0;
int mode = O_CREAT|O_RDWR; int mode = O_CREAT|O_RDWR;

View file

@ -51,7 +51,7 @@
#include "compression/code.h" #include "compression/code.h"
encoding_unit::encoding_unit(ostring* w, unsigned int f) : encoding_unit::encoding_unit(ostring* w, unsigned int f) :
word(w), freq(f), code(0) word(w), freq(f), code(0), bits(0), leaf_htr_node(NULL)
{ {
} }

View file

@ -32,6 +32,7 @@ CC_Tokenizer::CC_Tokenizer( const CC_String &s )
*((char *) memcpy(str_, s.data(), len) + len) = '\0'; *((char *) memcpy(str_, s.data(), len) + len) = '\0';
current_ptr = str_; current_ptr = str_;
touched = FALSE; touched = FALSE;
strtok_buf = 0;
} }

View file

@ -40,7 +40,7 @@ char *Exception::g_next_avail = Exception::g_temp_space;
// ///////////////////////////////////////////////////////////////// // /////////////////////////////////////////////////////////////////
Exception::Exception() Exception::Exception()
: f_thrown(0), f_thrown_as_pointer(1), f_temporary(0), f_line(0) : f_thrown(0), f_thrown_as_pointer(1), f_temporary(0), f_line(0), f_file(NULL), f_previous_exception(NULL)
{ {
PRINTF (("Constructed Exception obj @ %p\n", this)); PRINTF (("Constructed Exception obj @ %p\n", this));
} }

View file

@ -63,7 +63,7 @@ void c_index::init_persistent_info(persistent_info* x)
set_mode(SWAP_ALLOWED, false); set_mode(SWAP_ALLOWED, false);
} }
c_index::c_index(c_code_t c_cd) : composite(c_cd) c_index::c_index(c_code_t c_cd) : composite(c_cd), v_inv_lists_hd(NULL)
{ {
v_cmp_selector = 0; v_cmp_selector = 0;
} }

View file

@ -62,6 +62,7 @@ mphf_index::mphf_index() : c_index(MPHF_INDEX_CODE), f_key_file_name(0)
v_mphf = 0; v_mphf = 0;
v_inv_lists_hd = 0; v_inv_lists_hd = 0;
v_key_loc_pair_out = NULL;
} }
void mphf_index::init_data_member(fast_mphf_handler* mp, void mphf_index::init_data_member(fast_mphf_handler* mp,

View file

@ -109,6 +109,8 @@ oid_t::oid_t(const char* source, Boolean ascii_format, Boolean swap_order)
} }
} }
v_e_code = 0;
/* /*
MESSAGE(cerr, "constr oid_t using a string"); MESSAGE(cerr, "constr oid_t using a string");
debug(cerr, v_c_code); debug(cerr, v_c_code);

View file

@ -51,7 +51,7 @@
#include "oliasdb/mark.h" #include "oliasdb/mark.h"
umark::umark() : mmdb_tuple(2, USER_MARK_CODE) umark::umark() : mmdb_tuple(2, USER_MARK_CODE), mark_set_hd_ptr(NULL)
{ {
} }

View file

@ -57,6 +57,7 @@
mark_base::mark_base(user_base::rw_flag_t rw) : mark_base::mark_base(user_base::rw_flag_t rw) :
user_base(MARK_SPEC, rw) user_base(MARK_SPEC, rw)
{ {
mark_set_hd = NULL;
} }
@ -67,8 +68,10 @@ mark_base::mark_base( const char* base_dir,
) : ) :
user_base(base_dir, base_nm, base_ds, MARK_SPEC, rw) user_base(base_dir, base_nm, base_ds, MARK_SPEC, rw)
{ {
if ( checking_status != SUCC ) if ( checking_status != SUCC ) {
mark_set_hd = NULL;
return; return;
}
desc* ptr = first_desc_ptr; desc* ptr = first_desc_ptr;

View file

@ -68,7 +68,7 @@ extern void schemarestart(FILE*);
extern FILE *schemain; extern FILE *schemain;
object_dict::object_dict() : object_dict::object_dict() :
v_dict(desc_name_eq, desc_name_ls), v_desc_ptr(0) v_dict(desc_name_eq, desc_name_ls), v_desc_ptr(0), v_last_desc_ptr(NULL)
{ {
v_db_path[0] = 0; v_db_path[0] = 0;
} }

View file

@ -76,6 +76,8 @@ store_desc::store_desc(const char* name) : desc(name)
if ( order_str == 0 ) if ( order_str == 0 )
throw(stringException("machine type not supported")); throw(stringException("machine type not supported"));
v_store_ptr = NULL;
} }
store_desc::store_desc(int tp, const char* comment): store_desc::store_desc(int tp, const char* comment):
@ -95,6 +97,8 @@ store_desc::store_desc(int tp, const char* comment):
if ( order_str == 0 ) if ( order_str == 0 )
throw(stringException("machine type not supported")); throw(stringException("machine type not supported"));
v_store_ptr = NULL;
} }
store_desc::~store_desc() store_desc::~store_desc()

View file

@ -59,6 +59,8 @@ xtime::xtime() :
v_cpu_stamp(0), v_elapsed_stamp(0) v_cpu_stamp(0), v_elapsed_stamp(0)
{ {
memset(&v_time_regs, 0, sizeof(struct tms));
memset(&v_tv, 0, sizeof(struct timeval));
} }
void xtime::stop(float &cpu_time, long &elp_time) void xtime::stop(float &cpu_time, long &elp_time)

View file

@ -35,7 +35,10 @@ public:
BookmarkEdit (UAS_Pointer<Mark> &mark) BookmarkEdit (UAS_Pointer<Mark> &mark)
: f_mark_ptr (mark), : f_mark_ptr (mark),
f_shell (NULL), f_shell (NULL),
f_modified (FALSE) f_modified (FALSE),
f_name_text(NULL),
f_notes_text(NULL),
f_wm_delete_callback(NULL)
{ {
MarkMgr::request ((UAS_Receiver<MarkMoved> *) this); MarkMgr::request ((UAS_Receiver<MarkMoved> *) this);
} }

View file

@ -121,7 +121,12 @@ GraphicAgent::GraphicAgent (UAS_Pointer<UAS_Common> &node_ptr,
f_panner_state (PANNER_NONE), f_panner_state (PANNER_NONE),
f_current_scale (100), f_current_scale (100),
f_scale_button(NULL), f_scale_button(NULL),
f_setcustom(0) f_setcustom(0),
f_panner(NULL),
f_pixmap_widget(NULL),
f_view_menu(NULL),
f_message_area(NULL),
f_custom_scale(NULL)
{ {
f_graphic->pixmap_graphic()->agent(this); f_graphic->pixmap_graphic()->agent(this);

View file

@ -133,7 +133,8 @@ HelpAgent *HelpAgent::g_help_agent;
// ///////////////////////////////////////////////////////////////// // /////////////////////////////////////////////////////////////////
HelpAgent::HelpAgent() HelpAgent::HelpAgent()
: f_helper (NULL) : f_helper (NULL),
f_appXrmDb(NULL)
{ {
Widget app_shell = window_system().toplevel(); Widget app_shell = window_system().toplevel();
f_cursor = create_help_cursor(app_shell); f_cursor = create_help_cursor(app_shell);

View file

@ -136,6 +136,21 @@ LibraryAgent::LibraryAgent()
f_tracking_hierarchy (NULL), f_tracking_hierarchy (NULL),
f_popped_down (TRUE), f_popped_down (TRUE),
f_close (NULL), f_close (NULL),
f_close_sensitive (FALSE) f_close_sensitive (FALSE),
f_doc_tree_view(),
f_copy(NULL),
f_detach(NULL),
f_detach2(NULL),
f_view(NULL),
f_view2(NULL),
f_print(NULL),
f_print2(NULL),
f_print_as(NULL),
f_remove(NULL),
f_remove2(NULL),
f_auto_track(NULL),
f_status_text(NULL),
f_scope_menu(NULL)
{ {
} }

View file

@ -220,7 +220,16 @@ MapButton::destroy()
MapAgent::MapAgent() MapAgent::MapAgent()
: f_shell (NULL), : f_shell (NULL),
f_onscreen (FALSE) f_onscreen (FALSE),
f_locked(FALSE),
f_map_mode(LOCAL_MODE),
f_porthole(NULL),
f_panner(NULL),
f_tree(NULL),
f_wm_delete_callback(NULL),
f_lock(NULL),
f_min_tree_width(0),
f_min_tree_height(0)
{ {
} }

View file

@ -84,7 +84,8 @@
MarkChooser::MarkChooser (Widget parent, xList<MarkCanvas *> &marks, MarkChooser::MarkChooser (Widget parent, xList<MarkCanvas *> &marks,
const char *title_key, const char *ok_key) const char *title_key, const char *ok_key)
: f_selected_item (-1), : f_selected_item (-1),
f_mark_list (&marks) f_mark_list (&marks),
f_done(FALSE)
{ {
create_ui (parent, title_key, ok_key); create_ui (parent, title_key, ok_key);
update_list(); update_list();

View file

@ -82,7 +82,8 @@
MarkListView::MarkListView() MarkListView::MarkListView()
: f_shell (NULL), : f_shell (NULL),
f_popped_up (FALSE) f_popped_up (FALSE),
f_selected_item(0)
{ {
} }

View file

@ -72,6 +72,10 @@ private: // variables
inline inline
MessageAgent::MessageAgent() MessageAgent::MessageAgent()
: f_dialog (NULL), : f_dialog (NULL),
f_exit_flag (False) f_exit_flag (False),
f_text(NULL),
f_real_parent(NULL),
f_popped_up(FALSE),
f_pressed_ok(FALSE)
{ {
} }

View file

@ -38,7 +38,7 @@ class NodeHistoryAgent : public WWL,
{ {
public: // functions public: // functions
NodeHistoryAgent() NodeHistoryAgent()
: f_shell (NULL), f_selected_item (0) : f_shell (NULL), f_selected_item (0), f_popped_up(FALSE)
{ } { }
virtual ~NodeHistoryAgent(); virtual ~NodeHistoryAgent();
void display(); void display();

View file

@ -40,7 +40,16 @@ class WXmPanedWindow;
class NodeListAgent : public Agent class NodeListAgent : public Agent
{ {
public: // functions public: // functions
NodeListAgent() {} NodeListAgent() :
f_shell(NULL),
f_form(NULL),
f_menu_bar(NULL),
f_list_form(NULL),
f_list(NULL),
f_panel(NULL),
f_pane(NULL),
f_wm_delete_callback(NULL)
{}
~NodeListAgent(); ~NodeListAgent();
void init (); void init ();

View file

@ -278,7 +278,7 @@ _DtHelpCreatePrintArea(Widget parent,
// ///////////////////////////////////////////////////////////////// // /////////////////////////////////////////////////////////////////
NodePrintAgent::NodePrintAgent() NodePrintAgent::NodePrintAgent()
: f_node_view_info (NULL), f_help_dsp_area(NULL) : f_node_view_info (NULL), f_help_dsp_area(NULL), f_form(NULL)
{ {
} }

View file

@ -787,7 +787,34 @@ NodeWindowAgent::NodeWindowAgent (u_int serial_no)
f_hscrollbar_offset(0), f_hscrollbar_offset(0),
f_graphic_segment(NULL), f_graphic_segment(NULL),
f_graphics_handler(NULL), f_graphics_handler(NULL),
f_close_sensitive(FALSE) f_close_sensitive(FALSE),
f_frame(NULL),
f_create_bmrk(NULL),
f_create_anno(NULL),
f_create_link(NULL),
f_move_mark(NULL),
f_edit_mark(NULL),
f_delete_mark(NULL),
f_detach_graphic(NULL),
f_detach_menu(NULL),
f_detach_button(NULL),
f_attach_button(NULL),
f_raise_button(NULL),
f_preview_menu(NULL),
f_preview_label(NULL),
f_default_menu(NULL),
f_clear_search_hits(NULL),
f_graphical_map(NULL),
f_search_form(NULL),
f_search_menu_button(NULL),
f_print(NULL),
f_print2(NULL),
f_print_as(NULL),
f_move_mark_sensitive(0),
f_last_access_time(0),
f_locked(FALSE),
f_wm_delete_callback(NULL)
{ {
UAS_Common::request ((UAS_Receiver<UAS_LibraryDestroyedMsg> *) this); UAS_Common::request ((UAS_Receiver<UAS_LibraryDestroyedMsg> *) this);

View file

@ -259,7 +259,7 @@ public:
PrefAgent::PrefAgent() PrefAgent::PrefAgent()
: f_shell(NULL) : f_shell(NULL), f_top_panel(NULL), f_timeout(NULL)
{ {
} }

View file

@ -93,6 +93,9 @@ protected: // variables
inline inline
SearchResultsAgent::SearchResultsAgent() SearchResultsAgent::SearchResultsAgent()
: f_work_proc_id (0), f_popped_up(FALSE) : f_work_proc_id (0), f_popped_up(FALSE), f_my_ale(NULL), f_retain(FALSE),
f_results(NULL), f_selected_item(0), f_retain_toggle(NULL), f_query_text(NULL),
f_hits_label(NULL), f_scope_label(NULL), f_docs_to_display(0), f_count(0),
f_scale(0), f_header_indent(0)
{ {
} }

View file

@ -77,6 +77,7 @@ extern char g_top_locator[];
UrlAgent::UrlAgent () { UrlAgent::UrlAgent () {
fShell = NULL; fShell = NULL;
fPrefWindow = 0; fPrefWindow = 0;
fPoppedUp = FALSE;
} }
UrlAgent::~UrlAgent () { UrlAgent::~UrlAgent () {

View file

@ -95,7 +95,8 @@ EnvMgr::EnvMgr() : f_argc(0),
f_lang(NULL), f_lang(NULL),
f_secondary(False), f_secondary(False),
f_verbose(False), f_verbose(False),
f_debug(False) f_debug(False),
f_autohelp(False)
{ {
const char* lang; const char* lang;
if ((lang = getenv("LC_ALL")) == NULL) if ((lang = getenv("LC_ALL")) == NULL)

View file

@ -70,7 +70,7 @@ struct HistoryDelete : public Destructable
struct HistoryAdd : public Destructable struct HistoryAdd : public Destructable
{ {
HistoryAdd (UAS_Pointer<UAS_Common> &node_ptr) HistoryAdd (UAS_Pointer<UAS_Common> &node_ptr)
: f_new_entry(node_ptr) { } : f_new_entry(node_ptr), f_moving(false) { }
UAS_Pointer<UAS_Common> f_new_entry; // UAS_Pointer to UAS_Common displayed. UAS_Pointer<UAS_Common> f_new_entry; // UAS_Pointer to UAS_Common displayed.
bool f_moving; // True if a moved entry. bool f_moving; // True if a moved entry.

View file

@ -98,7 +98,8 @@ class Graphic: public UAS_Base {
fPixmap (0), fPixmap (0),
fDetachedPixmap (0), fDetachedPixmap (0),
fDetached (0), fDetached (0),
fObj (doc->create_embedded_object (locator)) { fObj (doc->create_embedded_object (locator)),
fagent(NULL) {
} }
~Graphic () { ~Graphic () {

View file

@ -66,6 +66,7 @@
LocalHistoryMgr::LocalHistoryMgr() LocalHistoryMgr::LocalHistoryMgr()
{ {
f_current = NULL; f_current = NULL;
f_first = NULL;
} }

View file

@ -33,7 +33,7 @@ class LocalHistoryEntry
{ {
private: // functions private: // functions
LocalHistoryEntry (UAS_Pointer<UAS_Common> &doc_ptr) LocalHistoryEntry (UAS_Pointer<UAS_Common> &doc_ptr)
: f_doc_ptr (doc_ptr), f_anchor (NULL) { } : f_doc_ptr (doc_ptr), f_anchor (NULL), f_previous(NULL), f_next(NULL) { }
LocalHistoryEntry() LocalHistoryEntry()
{ } { }
~LocalHistoryEntry(); ~LocalHistoryEntry();

View file

@ -64,6 +64,8 @@ WorkspaceMgr::WorkspaceMgr()
DtWsmAddCurrentWorkspaceCallback( window_system().toplevel(), DtWsmAddCurrentWorkspaceCallback( window_system().toplevel(),
(DtWsmWsChangeProc) PrimaryWorkspace_cb, this ); (DtWsmWsChangeProc) PrimaryWorkspace_cb, this );
f_actual = 0;
} }

View file

@ -236,7 +236,11 @@ CanvasRenderer::CanvasRenderer(int font_scale)
f_link_idx (-1), f_link_idx (-1),
f_font_scale(font_scale), f_font_scale(font_scale),
fBogusSymbol(gElemSymTab->intern("%BOGUS")), fBogusSymbol(gElemSymTab->intern("%BOGUS")),
f_level(0) f_level(0),
f_vcc(0),
f_current_container(NULL),
f_current_displayable(NULL),
f_default_features(NULL)
{ {
// make symbols // make symbols
for ( int i=0; i < REND_SYMBOLS; i++) for ( int i=0; i < REND_SYMBOLS; i++)

View file

@ -64,6 +64,8 @@ AppPrintData::AppPrintData()
f_print_shell = NULL; f_print_shell = NULL;
f_print_only = FALSE; f_print_only = FALSE;
f_outline_element = NULL; f_outline_element = NULL;
f_print_list = NULL;
f_pshell_parent = NULL;
}; };
//--------- Destructors ---------------------------------- //--------- Destructors ----------------------------------

View file

@ -226,7 +226,10 @@ public:
Shell_Info (Widget w) Shell_Info (Widget w)
: f_shell (w), : f_shell (w),
f_size_hints (NULL), f_size_hints (NULL),
f_restore (False) f_restore (False),
f_has_size_hints(False),
f_iconic(False),
f_has_wm_state(False)
{ } { }
public: public:
@ -252,7 +255,14 @@ WindowSystem::WindowSystem (int &argc, char *argv[])
f_shell_list (20), f_shell_list (20),
f_cursor_stack_pos(-1), f_cursor_stack_pos(-1),
f_dtinfo_font(NULL), f_dtinfo_font(NULL),
f_dtinfo_space_font(NULL) f_dtinfo_space_font(NULL),
f_print_display(NULL),
f_default_print_pixmap(0),
f_print_defpix_width(0),
f_print_defpix_height(0),
f_detached_width(0),
f_detached_height(0),
f_print_screen(NULL)
{ {
f_argc = &argc; f_argc = &argc;
f_argv = argv; f_argv = argv;

View file

@ -105,7 +105,8 @@ QueryEditor::QueryEditor(UAS_SearchEngine& search_engine)
: f_query (NULL), : f_query (NULL),
f_query_view (NULL), f_query_view (NULL),
f_shell (NULL), f_shell (NULL),
f_null_terms (0) f_null_terms (0),
f_min_term_width(0)
{ {
f_query_editor = this; f_query_editor = this;

View file

@ -52,7 +52,8 @@ UAS_BookcaseEntry::UAS_BookcaseEntry(UAS_Pointer<UAS_Common> &bookcase,
int searchable) int searchable)
: f_searchable(searchable), : f_searchable(searchable),
f_name(0), f_name(0),
f_base_num(-1) f_base_num(-1),
f_infolib_num(0)
{ {
int len; int len;

View file

@ -32,7 +32,7 @@
#include "StyleSheet/FeatureValue.h" #include "StyleSheet/FeatureValue.h"
Tml_TextRenderer::Tml_TextRenderer(ostringstream &ostr, UAS_SearchZones &zones) : Tml_TextRenderer::Tml_TextRenderer(ostringstream &ostr, UAS_SearchZones &zones) :
f_ostr(ostr), f_zones(zones) f_ostr(ostr), f_zones(zones), f_current_level(0)
{ {
} }

View file

@ -17,7 +17,7 @@ template<class K, class V> class DictLink
DictLink<K,V>* pre; DictLink<K,V>* pre;
DictLink<K,V>* suc; DictLink<K,V>* suc;
DictLink(const K& k, const V& v) : key(k), value(v) { }; DictLink(const K& k, const V& v) : key(k), value(v), pre(NULL), suc(NULL) { };
~DictLink() { if (suc) delete suc; } // delete all links recursively ~DictLink() { if (suc) delete suc; } // delete all links recursively
}; };

View file

@ -89,12 +89,15 @@ BookCaseTask::BookCaseTask(const char *infolib)
if ( !Dispatch::RunTocGenOnly() ) { if ( !Dispatch::RunTocGenOnly() ) {
style = new StyleTaskDB(this); style = new StyleTaskDB(this);
addSubTask(style); addSubTask(style);
} else {
style = NULL;
} }
book = new BookTask(this); book = new BookTask(this);
addSubTask(book); addSubTask(book);
f_style = NULL; f_style = NULL;
f_search_storage = NULL;
} }

View file

@ -46,6 +46,9 @@ public:
if ( this != &t ) { if ( this != &t ) {
this->level = t.level; this->level = t.level;
this->Buf = t.Buf; this->Buf = t.Buf;
} else {
this->level = -1;
this->Buf = NULL;
} }
} }
Rec & operator=( Rec &t ) { Rec & operator=( Rec &t ) {

View file

@ -90,6 +90,8 @@ FirstOf::FirstOf( const Token &t,
} }
} }
elist = NULL;
} }
//-------------------------------------------------------------------------- //--------------------------------------------------------------------------

View file

@ -269,6 +269,8 @@ NodeData::NodeData( NodeTask *parent, const Token &t)
#endif #endif
write_start_tag ( t, NodeBuffer ); write_start_tag ( t, NodeBuffer );
internal_buffer = NULL;
} }
//--------------------------------------------------------------------- //---------------------------------------------------------------------

View file

@ -78,6 +78,7 @@ StyleTask::StyleTask()
f_buffer = NULL; f_buffer = NULL;
f_pathbuf = NULL; f_pathbuf = NULL;
f_locator = NULL; f_locator = NULL;
f_dataMode = inPath;
feature_depth = new Stack<int>; feature_depth = new Stack<int>;

View file

@ -56,6 +56,9 @@ Token::Token()
entity_defn = NULL; entity_defn = NULL;
tokType=TK_INVALID; tokType=TK_INVALID;
f_olaf = -1; f_olaf = -1;
f_level = 0;
f_file = NULL;
f_line = 0;
#ifdef FISH_DEBUG #ifdef FISH_DEBUG
DBUG_VOID_RETURN; DBUG_VOID_RETURN;