<feed xmlns='http://www.w3.org/2005/Atom'>
<title>platform_system_netd/libnetdutils, branch master</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/'/>
<entry>
<title>[LSC] Add LOCAL_LICENSE_KINDS to system/netd</title>
<updated>2021-02-12T23:38:42+00:00</updated>
<author>
<name>Bob Badour</name>
<email>bbadour@google.com</email>
</author>
<published>2021-02-12T23:38:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=4114d1a05073315df2e012429585fc45f1de3384'/>
<id>4114d1a05073315df2e012429585fc45f1de3384</id>
<content type='text'>
Added SPDX-license-identifier-Apache-2.0 to:
  Android.bp
  bpf_progs/Android.bp
  client/Android.bp
  libnetdbpf/Android.bp
  libnetdutils/Android.bp
  netutils_wrappers/Android.bp
  server/Android.bp
  tests/Android.bp
  tests/benchmarks/Android.bp

Bug: 68860345
Bug: 151177513
Bug: 151953481

Test: m all

Exempt-From-Owner-Approval: janitorial work
Change-Id: Ie250cbab6e287585b945160cba4c2a764778ebc4
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Added SPDX-license-identifier-Apache-2.0 to:
  Android.bp
  bpf_progs/Android.bp
  client/Android.bp
  libnetdbpf/Android.bp
  libnetdutils/Android.bp
  netutils_wrappers/Android.bp
  server/Android.bp
  tests/Android.bp
  tests/benchmarks/Android.bp

Bug: 68860345
Bug: 151177513
Bug: 151953481

Test: m all

Exempt-From-Owner-Approval: janitorial work
Change-Id: Ie250cbab6e287585b945160cba4c2a764778ebc4
</pre>
</div>
</content>
</entry>
<entry>
<title>Move OperationLimiter to DnsResolver</title>
<updated>2020-11-27T11:59:03+00:00</updated>
<author>
<name>Bernie Innocenti</name>
<email>codewiz@google.com</email>
</author>
<published>2020-11-27T11:59:03+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=86a755f647562f203bd27b8771aa05bc91ce3f77'/>
<id>86a755f647562f203bd27b8771aa05bc91ce3f77</id>
<content type='text'>
OperationLimiter logically belongs with its only customer, DnsResolver.

I added OperationLimiter to libnetdutils for code sharing, but it turns
out netd doesn't have any need for this because it doesn't include other
threaded servers.

Change-Id: If8de486c53875a17440f368e53dd7ad2af1e5e57
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
OperationLimiter logically belongs with its only customer, DnsResolver.

I added OperationLimiter to libnetdutils for code sharing, but it turns
out netd doesn't have any need for this because it doesn't include other
threaded servers.

Change-Id: If8de486c53875a17440f368e53dd7ad2af1e5e57
</pre>
</div>
</content>
</entry>
<entry>
<title>Add an IPPrefix::forString parsing function.</title>
<updated>2020-05-18T16:59:06+00:00</updated>
<author>
<name>Lorenzo Colitti</name>
<email>lorenzo@google.com</email>
</author>
<published>2020-05-18T16:23:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=f0db40813a9e47da137b1d983da4010805f89f13'/>
<id>f0db40813a9e47da137b1d983da4010805f89f13</id>
<content type='text'>
This provides a way to parse IP prefixes from strings, and
addresses a TODO. Instead of reusing NetdConstants' parsePrefix
function, which it cannot depend on, it implements the parsing
using the IPAddress class in the same file and standard string
methods.

Bug: 153694684
Test: new unit tests
Change-Id: I47b29c8525f3c0236e48b4bb8dbebb223e83bc31
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This provides a way to parse IP prefixes from strings, and
addresses a TODO. Instead of reusing NetdConstants' parsePrefix
function, which it cannot depend on, it implements the parsing
using the IPAddress class in the same file and standard string
methods.

Bug: 153694684
Test: new unit tests
Change-Id: I47b29c8525f3c0236e48b4bb8dbebb223e83bc31
</pre>
</div>
</content>
</entry>
<entry>
<title>Set min_sdk_version to be part of mainline modules</title>
<updated>2020-05-01T17:09:52+00:00</updated>
<author>
<name>Jooyung Han</name>
<email>jooyung@google.com</email>
</author>
<published>2020-04-16T09:48:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=3c2225517c4cc33d8016ce286d0c04374feb6f74'/>
<id>3c2225517c4cc33d8016ce286d0c04374feb6f74</id>
<content type='text'>
Modules contributing mainline modules (APK/APEX) should set
min_sdk_version as well as apex_available.

For now setting min_sdk_version doesn't change build outputs.
But build-time checks will be added soon.

Bug: 152655956
Test: m
Change-Id: I92c78b0930a14f2f5830d6f0873996c17d4ee44b
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Modules contributing mainline modules (APK/APEX) should set
min_sdk_version as well as apex_available.

For now setting min_sdk_version doesn't change build outputs.
But build-time checks will be added soon.

Bug: 152655956
Test: m
Change-Id: I92c78b0930a14f2f5830d6f0873996c17d4ee44b
</pre>
</div>
</content>
</entry>
<entry>
<title>Set apex_available property</title>
<updated>2020-04-08T14:40:11+00:00</updated>
<author>
<name>Jiyong Park</name>
<email>jiyong@google.com</email>
</author>
<published>2020-04-08T14:40:11+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=3144db91ba989f6744eba50e8897009a02bb26e3'/>
<id>3144db91ba989f6744eba50e8897009a02bb26e3</id>
<content type='text'>
The marked library(ies) were available to the APEXes via the
hand-written whitelist in build/soong/apex/apex.go. Trying to remove the
whitelist by adding apex_available property to the Android.bp of the
libraries.

Bug: 150999716
Test: m
Change-Id: I274feb4542624b348999e7c5f189f8606c6cd63d
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The marked library(ies) were available to the APEXes via the
hand-written whitelist in build/soong/apex/apex.go. Trying to remove the
whitelist by adding apex_available property to the Android.bp of the
libraries.

Bug: 150999716
Test: m
Change-Id: I274feb4542624b348999e7c5f189f8606c6cd63d
</pre>
</div>
</content>
</entry>
<entry>
<title>FdTest - fix some compile time warnings</title>
<updated>2020-04-02T05:17:29+00:00</updated>
<author>
<name>Maciej Żenczykowski</name>
<email>maze@google.com</email>
</author>
<published>2020-04-02T00:15:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=eb4a339f0014bd46399ce23ad63cbd4de6c6eba5'/>
<id>eb4a339f0014bd46399ce23ad63cbd4de6c6eba5</id>
<content type='text'>
This fixes a number of warnings of the form:

  [ 98% 1866/1904] //system/netd/libnetdutils:netdutils_test clang-tidy FdTest.cpp [x86]
  //aosp/system/netd/libnetdutils/FdTest.cpp:105:35: warning: 'u1' used after it was moved [bugprone-use-after-move]
          EXPECT_FALSE(isWellFormed(u1));
                                    ^
  //aosp/system/netd/libnetdutils/FdTest.cpp:104:18: note: move occurred here
          UniqueFd u2(std::move(u1));
                   ^

based on searching for "EXPECT_FALSE(isWellFormed(" in this file.

The warnings are correct, but this is precisely testing for this case,
so just disable the warnings.

Test: builds
Bug: 153035880
Signed-off-by: Maciej Żenczykowski &lt;maze@google.com&gt;
Change-Id: Ie9cdd64e3acf168f755abfc80654c89e6e36d588
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This fixes a number of warnings of the form:

  [ 98% 1866/1904] //system/netd/libnetdutils:netdutils_test clang-tidy FdTest.cpp [x86]
  //aosp/system/netd/libnetdutils/FdTest.cpp:105:35: warning: 'u1' used after it was moved [bugprone-use-after-move]
          EXPECT_FALSE(isWellFormed(u1));
                                    ^
  //aosp/system/netd/libnetdutils/FdTest.cpp:104:18: note: move occurred here
          UniqueFd u2(std::move(u1));
                   ^

based on searching for "EXPECT_FALSE(isWellFormed(" in this file.

The warnings are correct, but this is precisely testing for this case,
so just disable the warnings.

Test: builds
Bug: 153035880
Signed-off-by: Maciej Żenczykowski &lt;maze@google.com&gt;
Change-Id: Ie9cdd64e3acf168f755abfc80654c89e6e36d588
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove CFI diagnostics mode</title>
<updated>2020-03-12T05:19:51+00:00</updated>
<author>
<name>Ken Chen</name>
<email>cken@google.com</email>
</author>
<published>2020-03-12T02:05:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=53b155d5d7113aebaab7172668e8e044210d8f3b'/>
<id>53b155d5d7113aebaab7172668e8e044210d8f3b</id>
<content type='text'>
CFI diagnostics sould be disabled since it introduces additional
overhead/complexity, and the diagnostics libraries haven't been analyzed
from a security perspective since they were primarily meant for testing.
The CFI will still work.

Bug: 146408702
Bug: 149427927
Change-Id: I4afb84fc9d64439f6660c44343abefa62f9fedf4
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
CFI diagnostics sould be disabled since it introduces additional
overhead/complexity, and the diagnostics libraries haven't been analyzed
from a security perspective since they were primarily meant for testing.
The CFI will still work.

Bug: 146408702
Bug: 149427927
Change-Id: I4afb84fc9d64439f6660c44343abefa62f9fedf4
</pre>
</div>
</content>
</entry>
<entry>
<title>netdutils: make {EXPECT, ASSERT}_OK for Status and Status-like objects</title>
<updated>2020-03-05T06:11:47+00:00</updated>
<author>
<name>Hungming Chen</name>
<email>nuccachen@google.com</email>
</author>
<published>2020-02-27T09:33:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=9d338eefc7a926f0f6ec0bc5ebd70f5f6a0d7deb'/>
<id>9d338eefc7a926f0f6ec0bc5ebd70f5f6a0d7deb</id>
<content type='text'>
Test: build, atest
Bug: 149963652
Change-Id: I397e693d488c4a955a40062b0aa93caee35eaa00
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Test: build, atest
Bug: 149963652
Change-Id: I397e693d488c4a955a40062b0aa93caee35eaa00
</pre>
</div>
</content>
</entry>
<entry>
<title>Convert system/netd to Result::ok()</title>
<updated>2020-02-05T18:58:30+00:00</updated>
<author>
<name>Bernie Innocenti</name>
<email>codewiz@google.com</email>
</author>
<published>2020-02-05T18:55:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=615fd7410a1c1a459c45cb610fe359bedcfb3cc1'/>
<id>615fd7410a1c1a459c45cb610fe359bedcfb3cc1</id>
<content type='text'>
No functionality changes, this is a mechanical cleanup.

Change-Id: I3181d34b52a05ed26296e2e0e72a5cdb8e955712
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
No functionality changes, this is a mechanical cleanup.

Change-Id: I3181d34b52a05ed26296e2e0e72a5cdb8e955712
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "Enable CFI (Control Flow Integrity)"</title>
<updated>2020-01-29T15:00:10+00:00</updated>
<author>
<name>Ken Chen</name>
<email>cken@google.com</email>
</author>
<published>2020-01-29T15:00:10+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_system_netd/commit/?id=c836c1a46c37e0c13a849eb9d463905f9ea72c3e'/>
<id>c836c1a46c37e0c13a849eb9d463905f9ea72c3e</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
