Skip to content

Commit

Permalink
Merge pull request rust-lang#908 from llogiq/master
Browse files Browse the repository at this point in the history
Update 0840-no-panic-in-c-string.md
  • Loading branch information
alexcrichton committed Mar 5, 2015
2 parents 8ce380a + 9907335 commit b906439
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions text/0840-no-panic-in-c-string.md
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,9 @@ reason for that exists; composition is preferred to adding function variants.
Longer function names, together with a less convenient return value, may deter
people from using the safer functions.

The panicky functions could also be renamed to `unpack_slice` and `unpack_vec`,
respectively, to highlight their conceptual proximity to `unpack`.

If the panicky behavior is preserved, plentiful possibilities for DoS attacks
and other unforeseen failures in the field may be introduced by code oblivious
to the input constraints.
Expand Down

0 comments on commit b906439

Please sign in to comment.