Negative height paramter's value leads to undefined behaviour

[alessio]

Follow-up of #4505
Issue: #4501

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
• Wrote tests
• Updated relevant documentation (docs/)
• Added a relevant changelog entry: clog add [section] [stanza] [message]
• rereviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[alexanderbez]

ACK

[codecov]

Codecov Report

Merging #4510 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4510      +/-   ##
==========================================
- Coverage   53.23%   53.22%   -0.01%     
==========================================
  Files         259      259              
  Lines       16203    16206       +3     
==========================================
+ Hits         8625     8626       +1     
- Misses       6932     6934       +2     
  Partials      646      646