mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] genext2fs: fix off-by-one in symlink creation
@ 2017-01-30 20:30 Uwe Kleine-König
  2017-02-08 15:52 ` Michael Olbrich
  0 siblings, 1 reply; 2+ messages in thread
From: Uwe Kleine-König @ 2017-01-30 20:30 UTC (permalink / raw)
  To: ptxdist

This fixes a failure when genext2fs tries to create a symlink where the target
has a length of exactly 60 bytes.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 ...x-inode-based-symlink-string-length-limit.patch | 39 ++++++++++++++++++++++
 patches/genext2fs-1.4.1/series                     |  4 +++
 2 files changed, 43 insertions(+)
 create mode 100644 patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
 create mode 100644 patches/genext2fs-1.4.1/series

diff --git a/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch b/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
new file mode 100644
index 000000000000..9a8182c0c9df
--- /dev/null
+++ b/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
@@ -0,0 +1,39 @@
+From: Alexander Stohr <Alexander.Stohr@gmx.de>
+Date: Sat, 31 Mar 2012 14:33:02 +0000
+Subject: [PATCH] fix inode based symlink string length limit
+
+Origin: http://genext2fs.cvs.sourceforge.net/viewvc/genext2fs/genext2fs/genext2fs.c?r1=1.95&r2=1.96
+[ukl: I'm annoyed because this patch is more complicated than necessary.
+Just doing s/>/>=/ in the 2nd line after the 2nd hunk has the same
+effect as the 2nd hunk without code duplication and dead code]
+---
+ genext2fs.c | 7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
+
+diff --git a/genext2fs.c b/genext2fs.c
+index 3087479511c4..bdc8aefc1fac 100644
+--- a/genext2fs.c
++++ b/genext2fs.c
+@@ -1415,9 +1415,10 @@ mklink_fs(filesystem *fs, uint32 parent_nod, const char *name, size_t size, uint
+ 	uint32 nod = mknod_fs(fs, parent_nod, name, FM_IFLNK | FM_IRWXU | FM_IRWXG | FM_IRWXO, uid, gid, 0, 0, ctime, mtime);
+ 	extend_blk(fs, nod, 0, - (int)get_nod(fs, nod)->i_blocks / INOBLK);
+ 	get_nod(fs, nod)->i_size = size;
+-	if(size <= 4 * (EXT2_TIND_BLOCK+1))
++	if(size < 4 * (EXT2_TIND_BLOCK+1))
+ 	{
+ 		strncpy((char*)get_nod(fs, nod)->i_block, (char*)b, size);
++		((char*)get_nod(fs, nod)->i_block)[size+1] = '\0';
+ 		return nod;
+ 	}
+ 	extend_blk(fs, nod, b, rndup(size, BLOCKSIZE) / BLOCKSIZE);
+@@ -1643,6 +1644,10 @@ add2fs_from_dir(filesystem *fs, uint32 this_nod, int squash_uids, int squash_per
+ 			switch(st.st_mode & S_IFMT)
+ 			{
+ 				case S_IFLNK:
++					if((st.st_mode & S_IFMT) == S_IFREG || st.st_size >= 4 * (EXT2_TIND_BLOCK+1))
++						stats->nblocks += (st.st_size + BLOCKSIZE - 1) / BLOCKSIZE;
++					stats->ninodes++;
++					break;
+ 				case S_IFREG:
+ 					if((st.st_mode & S_IFMT) == S_IFREG || st.st_size > 4 * (EXT2_TIND_BLOCK+1))
+ 						stats->nblocks += (st.st_size + BLOCKSIZE - 1) / BLOCKSIZE;
diff --git a/patches/genext2fs-1.4.1/series b/patches/genext2fs-1.4.1/series
new file mode 100644
index 000000000000..03d224b752ef
--- /dev/null
+++ b/patches/genext2fs-1.4.1/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-fix-inode-based-symlink-string-length-limit.patch
+# 09c5ef697ea11b2119735d1468222726  - git-ptx-patches magic
-- 
2.11.0


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [ptxdist] [PATCH] genext2fs: fix off-by-one in symlink creation
  2017-01-30 20:30 [ptxdist] [PATCH] genext2fs: fix off-by-one in symlink creation Uwe Kleine-König
@ 2017-02-08 15:52 ` Michael Olbrich
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Olbrich @ 2017-02-08 15:52 UTC (permalink / raw)
  To: ptxdist

On Mon, Jan 30, 2017 at 09:30:08PM +0100, Uwe Kleine-König wrote:
> This fixes a failure when genext2fs tries to create a symlink where the target
> has a length of exactly 60 bytes.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

Thanks, applied,

Michael

> ---
>  ...x-inode-based-symlink-string-length-limit.patch | 39 ++++++++++++++++++++++
>  patches/genext2fs-1.4.1/series                     |  4 +++
>  2 files changed, 43 insertions(+)
>  create mode 100644 patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
>  create mode 100644 patches/genext2fs-1.4.1/series
> 
> diff --git a/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch b/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
> new file mode 100644
> index 000000000000..9a8182c0c9df
> --- /dev/null
> +++ b/patches/genext2fs-1.4.1/0001-fix-inode-based-symlink-string-length-limit.patch
> @@ -0,0 +1,39 @@
> +From: Alexander Stohr <Alexander.Stohr@gmx.de>
> +Date: Sat, 31 Mar 2012 14:33:02 +0000
> +Subject: [PATCH] fix inode based symlink string length limit
> +
> +Origin: http://genext2fs.cvs.sourceforge.net/viewvc/genext2fs/genext2fs/genext2fs.c?r1=1.95&r2=1.96
> +[ukl: I'm annoyed because this patch is more complicated than necessary.
> +Just doing s/>/>=/ in the 2nd line after the 2nd hunk has the same
> +effect as the 2nd hunk without code duplication and dead code]
> +---
> + genext2fs.c | 7 ++++++-
> + 1 file changed, 6 insertions(+), 1 deletion(-)
> +
> +diff --git a/genext2fs.c b/genext2fs.c
> +index 3087479511c4..bdc8aefc1fac 100644
> +--- a/genext2fs.c
> ++++ b/genext2fs.c
> +@@ -1415,9 +1415,10 @@ mklink_fs(filesystem *fs, uint32 parent_nod, const char *name, size_t size, uint
> + 	uint32 nod = mknod_fs(fs, parent_nod, name, FM_IFLNK | FM_IRWXU | FM_IRWXG | FM_IRWXO, uid, gid, 0, 0, ctime, mtime);
> + 	extend_blk(fs, nod, 0, - (int)get_nod(fs, nod)->i_blocks / INOBLK);
> + 	get_nod(fs, nod)->i_size = size;
> +-	if(size <= 4 * (EXT2_TIND_BLOCK+1))
> ++	if(size < 4 * (EXT2_TIND_BLOCK+1))
> + 	{
> + 		strncpy((char*)get_nod(fs, nod)->i_block, (char*)b, size);
> ++		((char*)get_nod(fs, nod)->i_block)[size+1] = '\0';
> + 		return nod;
> + 	}
> + 	extend_blk(fs, nod, b, rndup(size, BLOCKSIZE) / BLOCKSIZE);
> +@@ -1643,6 +1644,10 @@ add2fs_from_dir(filesystem *fs, uint32 this_nod, int squash_uids, int squash_per
> + 			switch(st.st_mode & S_IFMT)
> + 			{
> + 				case S_IFLNK:
> ++					if((st.st_mode & S_IFMT) == S_IFREG || st.st_size >= 4 * (EXT2_TIND_BLOCK+1))
> ++						stats->nblocks += (st.st_size + BLOCKSIZE - 1) / BLOCKSIZE;
> ++					stats->ninodes++;
> ++					break;
> + 				case S_IFREG:
> + 					if((st.st_mode & S_IFMT) == S_IFREG || st.st_size > 4 * (EXT2_TIND_BLOCK+1))
> + 						stats->nblocks += (st.st_size + BLOCKSIZE - 1) / BLOCKSIZE;
> diff --git a/patches/genext2fs-1.4.1/series b/patches/genext2fs-1.4.1/series
> new file mode 100644
> index 000000000000..03d224b752ef
> --- /dev/null
> +++ b/patches/genext2fs-1.4.1/series
> @@ -0,0 +1,4 @@
> +# generated by git-ptx-patches
> +#tag:base --start-number 1
> +0001-fix-inode-based-symlink-string-length-limit.patch
> +# 09c5ef697ea11b2119735d1468222726  - git-ptx-patches magic
> -- 
> 2.11.0
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-02-08 15:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-30 20:30 [ptxdist] [PATCH] genext2fs: fix off-by-one in symlink creation Uwe Kleine-König
2017-02-08 15:52 ` Michael Olbrich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox