Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove references to a function removed long ago #76

Merged

Conversation

mikeytdisco
Copy link
Contributor

@mikeytdisco mikeytdisco commented Apr 4, 2023

Description

Code comments and the library DfciUiSupportLibNull reference a function no longer used. This removes the dead code from the NULL library, and updates comments in the active code. Fixes #75

For each item, place an "x" in between [ and ] if true. Example: [x].
(you can also check items in the GitHub UI)

  • Impacts functionality?

  • Impacts security?

  • Breaking change?

  • Includes tests?

  • Includes documentation?

How This Was Tested

Not tested

Integration Instructions

N/A

@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Apr 4, 2023
@spbrogan spbrogan force-pushed the personal/mikey/RemoveOldFunction branch from 6d3250b to 96793b2 Compare April 6, 2023 17:39
@spbrogan spbrogan merged commit 426e8d5 into microsoft:main Apr 6, 2023
@mikeytdisco mikeytdisco deleted the personal/mikey/RemoveOldFunction branch April 7, 2023 17:53
ProjectMuBot referenced this pull request in microsoft/mu_tiano_platforms Apr 20, 2023
…rom `2.0.2` to `2.3.1`.

Introduces 6 new commits.

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>
ProjectMuBot referenced this pull request in microsoft/mu_tiano_platforms Apr 20, 2023
…rom `2.0.2` to `2.3.1`.

Introduces 6 new commits.

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>
ProjectMuBot referenced this pull request in makubacki/mu_tiano_platforms Apr 20, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
ProjectMuBot referenced this pull request in makubacki/mu_tiano_platforms Apr 20, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
ProjectMuBot referenced this pull request in makubacki/mu_tiano_platforms Apr 20, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
ProjectMuBot referenced this pull request in makubacki/mu_tiano_platforms Apr 20, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
ProjectMuBot referenced this pull request in makubacki/mu_tiano_platforms Apr 20, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
ProjectMuBot referenced this pull request in microsoft/mu_tiano_platforms Apr 21, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
makubacki referenced this pull request in microsoft/mu_tiano_platforms Apr 21, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
makubacki referenced this pull request in microsoft/mu_tiano_platforms Apr 22, 2023
Introduces 6 new commits in [Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a> [Bug] Revert invalid change which breaks DFCI Refresh from network. (<a href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a> Make DFCI Lock Var Runtime accessible (<a href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a> Revert VarPolicy locking change (<a href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a> Fix memory leak (<a href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a> Fix duplicate name error when adding second firewall rule (<a href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a> Remove references to a function removed long ago (<a href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
makubacki referenced this pull request in microsoft/mu_tiano_platforms Apr 22, 2023
Bumps Features/DFCI from `2.0.2` to `2.3.1`

Introduces 6 new commits in
[Features/DFCI](https://github.com/microsoft/mu_feature_dfci).

<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/3b324ef716d609367e32480c96bdbbaf28c1f581">3b324e</a>
[Bug] Revert invalid change which breaks DFCI Refresh from network. (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/61">#61</a>)</li>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/4bf67f28993390a8fbdbd1522ffb838c3ab8c236">4bf67f</a>
Make DFCI Lock Var Runtime accessible (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/64">#64</a>)</li>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/0fc28dd31eaa21ff68a373dd1da40e42954ba46a">0fc28d</a>
Revert VarPolicy locking change (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/70">#70</a>)</li>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/769a6091b42e44b69195dc72f30c38d4bd7782df">769a60</a>
Fix memory leak (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/66">#66</a>)</li>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/4cd4265bb0431ea341f2cdee6495b5c2c8a8e904">4cd426</a>
Fix duplicate name error when adding second firewall rule (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/73">#73</a>)</li>
<li><a
href="https://github.com/microsoft/mu_feature_dfci/commit/426e8d513de072734935d7d38b5dd9aac606e0ba">426e8d</a>
Remove references to a function removed long ago (<a
href="https://github.com/microsoft/mu_feature_dfci/pull/76">#76</a>)</li>
</ul>
</details>

Signed-off-by: Project Mu Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:non-functional Does not have a functional impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Code comments an a NULL Library reference non existing code
3 participants