diff options
| author | Utkarsh Saxena <usaxena@codeaurora.org> | 2016-07-21 12:21:46 +0530 |
|---|---|---|
| committer | Utkarsh Saxena <usaxena@codeaurora.org> | 2016-07-21 12:21:46 +0530 |
| commit | edfee472bb22b6aa338a9efb42c37ae8ec55a550 (patch) | |
| tree | 6169655faa4e564842e328b08d2744405043e23e /ipacm/src/IPACM_Config.cpp | |
| parent | bf2f775ad26f0591370423d1276e310b405edd40 (diff) | |
| download | ipacfg-mgr-edfee472bb22b6aa338a9efb42c37ae8ec55a550.tar.gz | |
ipacm: Buffer overflow
Use strlcpy instead of memcpy to make sure
the NULL termination of this¬>ipa_virtual_iface_name
variable
Change-Id: I4057bd50a6ba46f7970472d7d42f518e49a0ab62
Diffstat (limited to 'ipacm/src/IPACM_Config.cpp')
| -rw-r--r-- | ipacm/src/IPACM_Config.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipacm/src/IPACM_Config.cpp b/ipacm/src/IPACM_Config.cpp index cb1d826..17d24a3 100644 --- a/ipacm/src/IPACM_Config.cpp +++ b/ipacm/src/IPACM_Config.cpp @@ -222,7 +222,7 @@ int IPACM_Config::Init(void) /* copy bridge interface name to ipacmcfg */ if( iface_table[i].if_cat == VIRTUAL_IF) { - memcpy(ipa_virtual_iface_name, iface_table[i].iface_name, sizeof(ipa_virtual_iface_name)); + strlcpy(ipa_virtual_iface_name, iface_table[i].iface_name, sizeof(ipa_virtual_iface_name)); IPACMDBG_H("ipa_virtual_iface_name(%s) \n", ipa_virtual_iface_name); } } |