Skip to content

Commit

Permalink
Merge #1020
Browse files Browse the repository at this point in the history
1020: Major cmsg cleanup r=asomers a=asomers

This PR fixes multiple bugs in the cmsg code.
Fixes #994 
Fixes #999 
Fixes #1013 

Co-authored-by: Alan Somers <asomers@gmail.com>
  • Loading branch information
bors[bot] and asomers committed Feb 14, 2019
2 parents 842f5ac + b5c4c7a commit a2fa282
Show file tree
Hide file tree
Showing 4 changed files with 457 additions and 481 deletions.
18 changes: 18 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,25 @@ This project adheres to [Semantic Versioning](http://semver.org/).
([#1002](https://github.com/nix-rust/nix/pull/1002))
### Changed
- `PollFd` event flags renamed to `PollFlags` ([#1024](https://github.com/nix-rust/nix/pull/1024/))
- `recvmsg` now returns an Iterator over `ControlMessageOwned` objects rather
than `ControlMessage` objects. This is sadly not backwards-compatible. Fix
code like this:
```rust
if let ControlMessage::ScmRights(&fds) = cmsg {
```

By replacing it with code like this:
```rust
if let ControlMessageOwned::ScmRights(fds) = cmsg {
```
([#1020](https://github.com/nix-rust/nix/pull/1020))
- Replaced `CmsgSpace` with the `cmsg_space` macro.
([#1020](https://github.com/nix-rust/nix/pull/1020))

### Fixed
- Fixed multiple bugs when using `sendmsg` and `recvmsg` with ancillary control messages
([#1020](https://github.com/nix-rust/nix/pull/1020))

### Removed

## [0.13.0] - 2019-01-15
Expand Down
Loading

0 comments on commit a2fa282

Please sign in to comment.