<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/fs/squashfs, branch v2021.01</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/fs/squashfs?h=v2021.01</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/fs/squashfs?h=v2021.01'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2020-12-02T21:21:58Z</updated>
<entry>
<title>fs/squashfs: sqfs_close/sqfs_read_sblk: set ctxt.sblk to NULL after free</title>
<updated>2020-12-02T21:21:58Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-24T17:07:52Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=7e932ac790b3615a67a3c24041c194aa748c0d98'/>
<id>urn:sha1:7e932ac790b3615a67a3c24041c194aa748c0d98</id>
<content type='text'>
This will prevent a double free error if sqfs_close() is called twice.

Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: implement exists() function</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:26Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=dd4866b43754b18f0c06672e341d93e16b8bf674'/>
<id>urn:sha1:dd4866b43754b18f0c06672e341d93e16b8bf674</id>
<content type='text'>
This permits to find a file and use the distro_bootcmd

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_read: remove buggy offset functionality</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:24Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=21b1b3bad58b50e5464b1bf016e7c96bf18ddb8d'/>
<id>urn:sha1:21b1b3bad58b50e5464b1bf016e7c96bf18ddb8d</id>
<content type='text'>
offset is the offset in the file read, not the offset in the destination
buffer.
If the offset is not null, this will lead to a memory corruption.
So, for now, we are returning an error if the offset is used.

Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_read: don't write beyond buffer size</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:23Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cbd5e40ede4e5c6aedce9475325bdf80b7fa839b'/>
<id>urn:sha1:cbd5e40ede4e5c6aedce9475325bdf80b7fa839b</id>
<content type='text'>
The length of the buffer wasn't taken into account when writing to the
given buffer.

Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_probe: use sqfs_decompressor_init() return value</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:22Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6d25bd3e9c1d317cdbd52a4c6bba11f912bb4d6a'/>
<id>urn:sha1:6d25bd3e9c1d317cdbd52a4c6bba11f912bb4d6a</id>
<content type='text'>
sqfs_decompressor_init() returns a value, so it's better to use it than
to force the return value to EINVAL (it could be ENOMEM)

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_probe: reset cur_dev/cur_part_info to NULL on error</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:21Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=56cf1ceee38435a0c105d40336d12d19e1db8b1d'/>
<id>urn:sha1:56cf1ceee38435a0c105d40336d12d19e1db8b1d</id>
<content type='text'>
Resetting the context on error will prevent some checks like:
if (!ctx.cur_dev)
To pass when the probe method has failed

Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_probe: fix possible memory leak on error</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:19Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ccd4c08a452b3703ee16ba730a84b7caadcff97a'/>
<id>urn:sha1:ccd4c08a452b3703ee16ba730a84b7caadcff97a</id>
<content type='text'>
If SquashFS magic number is invalid, there's a memory leak.

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_read: fix memory leak on finfo.blk_sizes</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:18Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=571b67ee1dcb989dc50575f14e87bfdef6bfd8db'/>
<id>urn:sha1:571b67ee1dcb989dc50575f14e87bfdef6bfd8db</id>
<content type='text'>
finfo.blk_sizes may not be freed in case of error in the for loop
Setting it to null and freeing it at the end makes prevents that from
happening.

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_get_abs_path: fix possible memory leak on error</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:17Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=33686804d29e676177d408f1ce047b8c35185ddb'/>
<id>urn:sha1:33686804d29e676177d408f1ce047b8c35185ddb</id>
<content type='text'>
if  sqfs_tokenize(rel_tokens, rc, rel); fails, the function exits
without freeing the array base_tokens.

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
<entry>
<title>fs/squashfs: sqfs_get_abs_path: fix error check</title>
<updated>2020-11-19T14:45:49Z</updated>
<author>
<name>Richard Genoud</name>
<email>richard.genoud@posteo.net</email>
</author>
<published>2020-11-03T11:11:16Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=53db0e24a8f981c1e303a1c745b8f0e28544096f'/>
<id>urn:sha1:53db0e24a8f981c1e303a1c745b8f0e28544096f</id>
<content type='text'>
the return value of sqfs_tokenize(rel_tokens, rc, rel); wasn't checked.
(but "ret" value was !)
This is obviouly a typo.

Reviewed-by: Joao Marcos Costa &lt;jmcosta944@gmail.com&gt;
Signed-off-by: Richard Genoud &lt;richard.genoud@posteo.net&gt;
</content>
</entry>
</feed>
