Gallopsled / pwntools / 2826

74%

push

travis-ci

Add cyclic default args to context (#1049)

* Add cyclic default args to context

Specifically, this adds context.cyclic_alphabet and context.cyclic_size

This helps cut down on writing "n=8" all the time for users who use 8-byte
subsequences on 64-bit architectures.

Exposing the default alphabet means that users can avoid "bad bytes" in
cyclic patterns which are generated by pwntools internally (e.g. ROP padding)

Some pwnlib.util.packing docstrings were changed to clarify that word_size is
in bits.

The doctests in pwnlib.util.cyclic were updated and extended to have more
and better doctests, as well as better exaples.

* Add an example showing an alphabet with NUL bytes

This is important because old code would use pack(..., "all") when an integer
value was provided, instead of using the correct width.

2251 of 5211 branches covered (43.2%)

28 of 28 new or added lines in 2 files covered. (100.0%)

7831 of 13531 relevant lines covered (57.87%)

0.58 hits per line