Skip to content

cmd: file policy import bug fix#507

Merged
tgraf merged 1 commit intomasterfrom
fixing-03-docker-test
Apr 4, 2017
Merged

cmd: file policy import bug fix#507
tgraf merged 1 commit intomasterfrom
fixing-03-docker-test

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Apr 4, 2017

While processing all files when importing from a directory, the policy
was not properly imported to the daemon. This was due the incorrect
assumption that a pointer's value could be assigned inside a function.
By passing a pointer of the pointer's value, it is possible to assign a
new value inside the function.

Signed-off-by: André Martins andre@cilium.io

While processing all files when importing from a directory, the policy
was not properly imported to the daemon. This was due the incorrect
assumption that a pointer's value could be assigned inside a function.
By passing a pointer of the pointer's value, it is possible to assign a
new value inside the function.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added the kind/bug This is a bug in the Cilium logic. label Apr 4, 2017
@aanm aanm requested a review from tgraf April 4, 2017 18:24
@tgraf tgraf merged commit 86c55f4 into master Apr 4, 2017
@tgraf tgraf deleted the fixing-03-docker-test branch April 4, 2017 18:49
tgraf pushed a commit that referenced this pull request Apr 4, 2017
While processing all files when importing from a directory, the policy
was not properly imported to the daemon. This was due the incorrect
assumption that a pointer's value could be assigned inside a function.
By passing a pointer of the pointer's value, it is possible to assign a
new value inside the function.

Backports: #507

Signed-off-by: André Martins <andre@cilium.io>
tgraf pushed a commit that referenced this pull request Apr 4, 2017
While processing all files when importing from a directory, the policy
was not properly imported to the daemon. This was due the incorrect
assumption that a pointer's value could be assigned inside a function.
By passing a pointer of the pointer's value, it is possible to assign a
new value inside the function.

Backports: #507

Signed-off-by: André Martins <andre@cilium.io>
yoursanonymous pushed a commit to yoursanonymous/cilium that referenced this pull request Jan 31, 2026
nddq pushed a commit to nddq/cilium that referenced this pull request Feb 17, 2026
…) (cilium#671)

Signed-off-by: Vipul Singh <singhvipul@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug This is a bug in the Cilium logic.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants