| cpio: Fix memory overrun on reading improperly created link records |
| |
| Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com> |
| |
| http://git.savannah.gnu.org/cgit/cpio.git/commit/?id=746f3ff670dcfcdd28fcc990e79cd6fccc7ae48d |
| |
| * src/copyin.c (get_link_name): New function. |
| (list_file, copyin_link): use get_link_name |
| |
| * tests/symlink-bad-length.at: New file. |
| * tests/symlink-long.at: New file. |
| * tests/Makefile.am: Add new files. |
| * tests/testsuite.at: Likewise. |
| |
| See http://lists.gnu.org/archive/html/bug-cpio/2014-11/msg00007.html |
| |
| Upstream-Status: Backport |
| |
| Signed-off-by: Sergey Poznyakoff <gray@gnu.org.ua> |
| |
| diff -Nurp cpio-2.11.orig/src/copyin.c cpio-2.11/src/copyin.c |
| --- cpio-2.11.orig/src/copyin.c 2010-02-15 18:02:23.000000000 +0800 |
| +++ cpio-2.11/src/copyin.c 2014-12-08 13:14:04.355547508 +0800 |
| @@ -126,6 +126,28 @@ tape_skip_padding (int in_file_des, off_ |
| } |
| |
| |
| +static char * |
| +get_link_name (struct cpio_file_stat *file_hdr, int in_file_des) |
| +{ |
| + off_t n = file_hdr->c_filesize + 1; |
| + char *link_name; |
| + |
| + if (n == 0 || n > SIZE_MAX) |
| + { |
| + error (0, 0, _("%s: stored filename length too big"), file_hdr->c_name); |
| + link_name = NULL; |
| + } |
| + else |
| + { |
| + link_name = xmalloc (n); |
| + tape_buffered_read (link_name, in_file_des, file_hdr->c_filesize); |
| + link_name[file_hdr->c_filesize] = '\0'; |
| + tape_skip_padding (in_file_des, file_hdr->c_filesize); |
| + } |
| + return link_name; |
| +} |
| + |
| + |
| static void |
| list_file(struct cpio_file_stat* file_hdr, int in_file_des) |
| { |
| @@ -136,21 +158,16 @@ list_file(struct cpio_file_stat* file_hd |
| { |
| if (archive_format != arf_tar && archive_format != arf_ustar) |
| { |
| - char *link_name = NULL; /* Name of hard and symbolic links. */ |
| - |
| - link_name = (char *) xmalloc ((unsigned int) file_hdr->c_filesize + 1); |
| - link_name[file_hdr->c_filesize] = '\0'; |
| - tape_buffered_read (link_name, in_file_des, file_hdr->c_filesize); |
| - long_format (file_hdr, link_name); |
| - free (link_name); |
| - tape_skip_padding (in_file_des, file_hdr->c_filesize); |
| - return; |
| + char *link_name = get_link_name (file_hdr, in_file_des); |
| + if (link_name) |
| + { |
| + long_format (file_hdr, link_name); |
| + free (link_name); |
| + } |
| } |
| else |
| - { |
| long_format (file_hdr, file_hdr->c_tar_linkname); |
| - return; |
| - } |
| + return; |
| } |
| else |
| #endif |
| @@ -650,10 +667,7 @@ copyin_link(struct cpio_file_stat *file_ |
| |
| if (archive_format != arf_tar && archive_format != arf_ustar) |
| { |
| - link_name = (char *) xmalloc ((unsigned int) file_hdr->c_filesize + 1); |
| - link_name[file_hdr->c_filesize] = '\0'; |
| - tape_buffered_read (link_name, in_file_des, file_hdr->c_filesize); |
| - tape_skip_padding (in_file_des, file_hdr->c_filesize); |
| + link_name = get_link_name (file_hdr, in_file_des); |
| } |
| else |
| { |
| diff -Nurp cpio-2.11.orig/tests/Makefile.am cpio-2.11/tests/Makefile.am |
| --- cpio-2.11.orig/tests/Makefile.am 2010-02-15 18:02:23.000000000 +0800 |
| +++ cpio-2.11/tests/Makefile.am 2014-12-08 13:14:49.931545727 +0800 |
| @@ -52,6 +52,8 @@ TESTSUITE_AT = \ |
| setstat04.at\ |
| setstat05.at\ |
| symlink.at\ |
| + symlink-bad-length.at\ |
| + symlink-long.at\ |
| version.at |
| |
| TESTSUITE = $(srcdir)/testsuite |
| diff -Nurp cpio-2.11.orig/tests/symlink-bad-length.at cpio-2.11/tests/symlink-bad-length.at |
| --- cpio-2.11.orig/tests/symlink-bad-length.at 1970-01-01 08:00:00.000000000 +0800 |
| +++ cpio-2.11/tests/symlink-bad-length.at 2014-12-08 13:17:45.979538847 +0800 |
| @@ -0,0 +1,49 @@ |
| +# Process this file with autom4te to create testsuite. -*- Autotest -*- |
| +# Copyright (C) 2014 Free Software Foundation, Inc. |
| + |
| +# This program is free software; you can redistribute it and/or modify |
| +# it under the terms of the GNU General Public License as published by |
| +# the Free Software Foundation; either version 3, or (at your option) |
| +# any later version. |
| + |
| +# This program is distributed in the hope that it will be useful, |
| +# but WITHOUT ANY WARRANTY; without even the implied warranty of |
| +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| +# GNU General Public License for more details. |
| + |
| +# You should have received a copy of the GNU General Public License |
| +# along with this program; if not, write to the Free Software |
| +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA |
| +# 02110-1301 USA. |
| + |
| +# Cpio v2.11 did segfault with badly set symlink length. |
| +# References: |
| +# http://lists.gnu.org/archive/html/bug-cpio/2014-11/msg00007.html |
| + |
| +AT_SETUP([symlink-bad-length]) |
| +AT_KEYWORDS([symlink-long copyout]) |
| + |
| +AT_DATA([ARCHIVE.base64], |
| +[x3EjAIBAtIEtJy8nAQAAAHRUYW0FAAAADQBGSUxFAABzb21lIGNvbnRlbnQKAMdxIwBgQ/+hLScv |
| +JwEAAAB0VEhuBQD/////TElOSwAARklMRcdxAAAAAAAAAAAAAAEAAAAAAAAACwAAAAAAVFJBSUxF |
| +UiEhIQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA |
| +AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA= |
| +]) |
| + |
| +AT_CHECK([ |
| +base64 -d ARCHIVE.base64 > ARCHIVE || AT_SKIP_TEST |
| +cpio -ntv < ARCHIVE |
| +test $? -eq 2 |
| +], |
| +[0], |
| +[-rw-rw-r-- 1 10029 10031 13 Nov 25 13:52 FILE |
| +],[cpio: LINK: stored filename length too big |
| +cpio: premature end of file |
| +]) |
| + |
| +AT_CLEANUP |
| diff -Nurp cpio-2.11.orig/tests/symlink-long.at cpio-2.11/tests/symlink-long.at |
| --- cpio-2.11.orig/tests/symlink-long.at 1970-01-01 08:00:00.000000000 +0800 |
| +++ cpio-2.11/tests/symlink-long.at 2014-12-08 13:17:57.219538408 +0800 |
| @@ -0,0 +1,46 @@ |
| +# Process this file with autom4te to create testsuite. -*- Autotest -*- |
| +# Copyright (C) 2014 Free Software Foundation, Inc. |
| + |
| +# This program is free software; you can redistribute it and/or modify |
| +# it under the terms of the GNU General Public License as published by |
| +# the Free Software Foundation; either version 3, or (at your option) |
| +# any later version. |
| + |
| +# This program is distributed in the hope that it will be useful, |
| +# but WITHOUT ANY WARRANTY; without even the implied warranty of |
| +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| +# GNU General Public License for more details. |
| + |
| +# You should have received a copy of the GNU General Public License |
| +# along with this program; if not, write to the Free Software |
| +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA |
| +# 02110-1301 USA. |
| + |
| +# Cpio v2.11.90 changed the way symlink name is read from archive. |
| +# References: |
| +# http://lists.gnu.org/archive/html/bug-cpio/2014-11/msg00007.html |
| + |
| +AT_SETUP([symlink-long]) |
| +AT_KEYWORDS([symlink-long copyout]) |
| + |
| +AT_CHECK([ |
| + |
| +# len(dirname) > READBUFSIZE |
| +dirname= |
| +for i in {1..52}; do |
| + dirname="xxxxxxxxx/$dirname" |
| + mkdir "$dirname" |
| +done |
| +ln -s "$dirname" x || AT_SKIP_TEST |
| + |
| +echo x | cpio -o > ar |
| +list=`cpio -tv < ar | sed 's|.*-> ||'` |
| +test "$list" = "$dirname" && echo success || echo fail |
| +], |
| +[0], |
| +[success |
| +],[2 blocks |
| +2 blocks |
| +]) |
| + |
| +AT_CLEANUP |
| diff -Nurp cpio-2.11.orig/tests/testsuite.at cpio-2.11/tests/testsuite.at |
| --- cpio-2.11.orig/tests/testsuite.at 2010-02-15 18:02:23.000000000 +0800 |
| +++ cpio-2.11/tests/testsuite.at 2014-12-08 13:15:13.515544805 +0800 |
| @@ -31,6 +31,8 @@ m4_include([version.at]) |
| |
| m4_include([inout.at]) |
| m4_include([symlink.at]) |
| +m4_include([symlink-bad-length.at]) |
| +m4_include([symlink-long.at]) |
| m4_include([interdir.at]) |
| |
| m4_include([setstat01.at]) |