fixes possible data truncation

[isandeep41]

Fixes #11461

---

This PR has:

• been self-reviewed.
• added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
• been tested in a test Druid cluster.

[lgtm-com]

This pull request fixes 1 alert when merging aabed1c into 8729b40 - view on LGTM.com

fixed alerts:

• 1 for User-controlled data in numeric cast

[asdf2014]

👍 LGTM

[lgtm-com]

This pull request fixes 1 alert when merging b0554b6 into 8729b40 - view on LGTM.com

fixed alerts:

• 1 for User-controlled data in numeric cast

[suneet-s]

@isandeep41 I see you marked integration tests have been added for this PR, but I don't see any in the delta. Have you been able to reproduce this scenario locally? What would it take to make this a test that shows the problem before your fix?

[isandeep41]

@suneet-s @asdf2014 I have added a unit test to catch the possible data truncation.

[lgtm-com]

This pull request fixes 1 alert when merging 7dd5faf into 167c452 - view on LGTM.com

fixed alerts:

• 1 for User-controlled data in numeric cast

[isandeep41]

@suneet-s I have updated the PR description, let me know if this looks good to you now

[FrankChen021]

👍. Since most of subclasses of UnivariateMathFunction overrides the eval(double) implementation, this change won't change the behaviors of these classes.