Comment 11 for bug 1511735

Revision history for this message
Ryan Harper (raharper) wrote :

Hi,

Thanks for the quick update. I've applied the 4 patches and built a new test package with the changes. The example test does indeed fail on the current version and passes on the new version as well.

I went looking for any other libnl3 tests we might run to catch any possible regression that might occur when accepting the patches I looked at the RH bug you referenced and found another testcase ( used to expose the issue. I ran that on the updated package and it passes tests a, b and c, but not d.

I examined the patch applied and it included more fixes, specifically:

Subject: [PATCH 09/10] socket: clear port when unable to generate local port

which the 'd' test which attempts to test by allocate more than 1024 local ports. Should a fix for that also be included?