Description of problem:
The latest pfmon package fails to build on ia64:

dear_hist_ia64.c: In function 'dear_hist_process_samples_normal':
dear_hist_ia64.c:189: error: 'csmpl' is used uninitialized in this function

from dear_hist_ia64.c:

        pfmon_smpl_desc_t *csmpl;
<.. snip ..>
        void *hash_desc = csmpl->data, *data;

csmpl really never is set to anything before we set a pointer to one of it's
fields.  Looking at where hash_desc is used I see that we really do need this to
be initialized to something.

Since csmpl is later initialized as:

csmpl = &sdesc->csmpl;

I _assume_ what was meant for the hash_desc initialization was to initialize
hash_desc after csmpl is set.  Please confirm this is what is intended (it
compiles but I don't know enough about pfmon to really test it).


Also, there was one other much more trivial case of using an uninitialized
variable in this patch:

inst_hist_ia64_old_smpl.c: In function 'inst_hist_show_results':
inst_hist_ia64_old_smpl.c:311: error: 'ns' may be used uninitialized in this
function

I will post a patch that fixes both issues.


Version-Release number of selected component (if applicable):
pfmon-3.3-0.080225.1.fc9

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info: