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

include: simplify strlcpy() #2700

Merged
merged 1 commit into from Oct 18, 2018
Merged

Conversation

brauner
Copy link
Member

@brauner brauner commented Oct 18, 2018

Signed-off-by: Christian Brauner christian.brauner@ubuntu.com

Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
@2xsec
Copy link
Contributor

2xsec commented Oct 18, 2018

@brauner I think all issues are related strlcpy() will be fixed with this patch. Actually, other cases are fixed in our side.

@2xsec
Copy link
Contributor

2xsec commented Oct 18, 2018

==lxc-attach==3249==ERROR: AddressSanitizer: global-buffer-overflow on address 0x00014f68 at pc 0xb613fa11 bp 0xbe3bf380 sp 0xbe3bf384
READ of size 4 at 0x00014f68 thread T0
#0 0xb613fa0f in strlcpy
#1 0xb60c7013 in lxc_log_init
#2 0x117ff in main
#3 0xb5eb3867 in __libc_start_main

@2xsec
Copy link
Contributor

2xsec commented Oct 18, 2018

==zone-launcher==1030==ERROR: AddressSanitizer: heap-buffer-overflow on address 0xb4560758 at pc 0xb60a7899 bp 0xbe16e390 sp 0xbe16e394
READ of size 4 at 0xb4560758 thread T0
#0 0xb60a7897 in strlcpy
#1 0xb605f94f in lxc_monitor_send_state
#2 0xb6068e6d in lxc_set_state
#3 0xb606a1b7 in lxc_init
#4 0xb60700af in __lxc_start
#5 0xb6070cc9 in lxc_start
#6 0xb6042577 in do_lxcapi_start
#7 0xb604351d in lxcapi_start
#8 0x4d1b31 in main
#9 0xb5cdc867 in __libc_start_main

@2xsec
Copy link
Contributor

2xsec commented Oct 18, 2018

@brauner Thanks. :)

@brauner
Copy link
Member Author

brauner commented Oct 18, 2018

Oh no worries, I'm guilty of using a complicated version of strlcpy(). Keeping it simple is usually better. :)

@Blub Blub merged commit 4665a84 into lxc:master Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants