fix array overflow issues found by David A.

Issues reported at
https://hypernews.cern.ch/HyperNews/CMS/get/swReleases/4108/1/1/1/1/1/1/1/1.html
and follow ups are mostly fixed here. 

On Sep 15, 2014, at 6:45 PM, Slava Krutelyov <notifications@github.com> wrote:

> In RecoEcal/EgammaCoreTools/plugins/EcalClusterEnergyUncertaintyObjectSpecific.cc:
> 
> >  	}
> >  	
> > -	if (TMath::Abs(eta)>2.5) iEtaSl = nBinsEta-1;
> > +	if (eta>EtaBins[nBinsEta-1]) iEtaSl = nBinsEta-1;
> 
> @lgray @Sam-Harper
> Shouldn't this be if (eta>=EtaBins[nBinsEta]) iEtaSl = nBinsEta-1; ?
> (note the possible equality since it's not covered in the logic above)
> 
> and, similalrly, below if (brem>=BremBins[nBinsBrem]) iBremSl = nBinsBrem-1;

I believe the effect is the same in either case given the loops just above

> 
> —
> Reply to this email directly or view it on GitHub.
>