September 2023 - Linux-stable-mirror

Include bac7a1fff792 ("lib/ubsan: remove returns-nonnull-attribute checks") into linux-4.14.y

by Lukas Bulwahn

Dear Andrey, dear Nick, dear Greg, dear Sasha, Compiling the kernel with UBSAN enabled and with gcc-8 and later fails when: commit 1e1b6d63d634 ("lib/string.c: implement stpcpy") is applied, and commit bac7a1fff792 ("lib/ubsan: remove returns-nonnull-attribute checks") is not applied. To reproduce, run: tuxmake -r docker -a arm64 -t gcc-13 -k allnoconfig --kconfig-add CONFIG_UBSAN=y It then fails with: aarch64-linux-gnu-ld: lib/string.o: in function `stpcpy': string.c:(.text+0x694): undefined reference to `__ubsan_handle_nonnull_return_v1' string.c:(.text+0x694): relocation truncated to fit: R_AARCH64_CALL26 against undefined symbol `__ubsan_handle_nonnull_return_v1' Below you find a complete list of architectures, compiler versions and kernel versions that I have tested with. As commit bac7a1fff792 ("lib/ubsan: remove returns-nonnull-attribute checks") is included in v4.16, and commit 1e1b6d63d634 ("lib/string.c: implement stpcpy") is included in v5.9, this is not an issue that can happen on any mainline release or the stable releases v4.19.y and later. In the v4.14.y branch, however, commit 1e1b6d63d634 ("lib/string.c: implement stpcpy") was included with v4.14.200 as commit b6d38137c19f and commit bac7a1fff792 ("lib/ubsan: remove returns-nonnull-attribute checks") from mainline was not included yet. Hence, this reported failure with UBSAN can be observed on v4.14.y with recent gcc versions. Greg, once checked and confirmed by Andrey or Nick, could you please include commit bac7a1fff792 ("lib/ubsan: remove returns-nonnull-attribute checks") into the linux-4.14.y branch? The commit applies directly, without any change, on v4.14.200 to v4.14.325. With that, future versions of v4.14.y will have a working UBSAN with the recent gcc compiler versions. Note: For any users, intending to run UBSAN on versions 4.14.200 to v4.14.325, e.g., for bisecting UBSAN-detected kernel bugs on the linux-4.14.y branch, they would simply need to apply commit bac7a1fff792 on those release versions. Appendix of my full testing record: For arm64 and x86-64 architecture, I tested this whole matrix of combinations of building v4.14.200, i.e., the first version that failed with the reported build failure and v4.14.325, i.e., the latest v4.14 release version at the time of writing. On v4.14.200 and on v4.14.325: x86_64: gcc-7: unsupported configuration (according to tuxmake) gcc-8: affected and resolved by cherry-picking bac7a1fff792 gcc-9: affected and resolved by cherry-picking bac7a1fff792 gcc-10: affected and resolved by cherry-picking bac7a1fff792 gcc-11: v4.14.200 fails with an unrelated build error on this compiler and arch v4.14.325 affected and resolved by cherry-picking bac7a1fff792 gcc-12: v4.14.200 fails with an unrelated build error on this compiler and arch v4.14.325 affected and resolved by cherry-picking bac7a1fff792 gcc-13: v4.14.200 fails with an unrelated build error on this compiler and arch v4.14.325 affected and resolved by cherry-picking bac7a1fff792 clang-9: unsupported configuration (according to tuxmake) clang-10: not affected, builds with and without cherry-picking bac7a1fff792 clang-17: not affected, builds with and without cherry-picking bac7a1fff792 arm64: gcc-7: unsupported configuration (according to tuxmake) gcc-8: affected and resolved by cherry-picking bac7a1fff792 gcc-9: affected and resolved by cherry-picking bac7a1fff792 gcc-10: affected and resolved by cherry-picking bac7a1fff792 gcc-11: affected and resolved by cherry-picking bac7a1fff792 gcc-12: affected and resolved by cherry-picking bac7a1fff792 gcc-13: affected and resolved by cherry-picking bac7a1fff792 clang-9: unsupported configuration (according to tuxmake) clang-10: not affected, builds with and without cherry-picking bac7a1fff792 clang-17: not affected, builds with and without cherry-picking bac7a1fff792 Best regards, Lukas

1 year, 3 months

3
2
0 0

Re: DRBD broken in kernel 6.5 and 6.5.1

by Jakub Kicinski

CC: David On Sat, 2 Sep 2023 22:31:06 +0200 Serguei Ivantsov wrote: > Hello, > > After upgrading the kernel to 6.5 the system can't connect to the peer > (6.4.11) anymore. > I checked 6.5.1 - same issue. > All previous kernels including 6.4.14 are working just fine. > Checking the 6.5 changelog, I found commit > 9ae440b8fdd6772b6c007fa3d3766530a09c9045 which mentioned some changes to > DRBD. I don't see anything obviously wrong here, maybe David has better ideas. Can you try this? diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c index 79ab532aabaf..c607d4304608 100644 --- a/drivers/block/drbd/drbd_main.c +++ b/drivers/block/drbd/drbd_main.c @@ -1539,8 +1539,6 @@ static int _drbd_send_page(struct drbd_peer_device *peer_device, struct page *pa int offset, size_t size, unsigned msg_flags) { struct socket *socket = peer_device->connection->data.socket; - struct msghdr msg = { .msg_flags = msg_flags, }; - struct bio_vec bvec; int len = size; int err = -EIO; @@ -1551,10 +1549,13 @@ static int _drbd_send_page(struct drbd_peer_device *peer_device, struct page *pa * __page_cache_release a page that would actually still be referenced * by someone, leading to some obscure delayed Oops somewhere else. */ if (!drbd_disable_sendpage && sendpage_ok(page)) - msg.msg_flags |= MSG_NOSIGNAL | MSG_SPLICE_PAGES; + msg_flags |= MSG_SPLICE_PAGES; + msg_flags |= MSG_NOSIGNAL; drbd_update_congested(peer_device->connection); do { + struct msghdr msg = { .msg_flags = msg_flags, }; + struct bio_vec bvec; int sent; bvec_set_page(&bvec, page, offset, len); > On the 6.5.X system I have the following in the kernel log (drbd_send_block() > failed): > > [ 2.473497] drbd: initialized. Version: 8.4.11 (api:1/proto:86-101) > > [ 2.475394] drbd: built-in > > [ 2.477254] drbd: registered as block device major 147 > > [ 7.421400] drbd drbd0: Starting worker thread (from drbdsetup-84 [3844]) > > [ 7.421509] drbd drbd0/0 drbd0: disk( Diskless -> Attaching ) > > [ 7.421552] drbd drbd0: Method to ensure write ordering: flush > > [ 7.421554] drbd drbd0/0 drbd0: max BIO size = 131072 > > [ 7.421557] drbd drbd0/0 drbd0: drbd_bm_resize called with capacity == > 1845173184 > > [ 7.428017] drbd drbd0/0 drbd0: resync bitmap: bits=230646648 > words=3603854 pages=7039 > > [ 7.467370] drbd0: detected capacity change from 0 to 1845173184 > > [ 7.467372] drbd drbd0/0 drbd0: size = 880 GB (922586592 KB) > > [ 7.486005] drbd drbd0/0 drbd0: recounting of set bits took additional 0 > jiffies > > [ 7.486010] drbd drbd0/0 drbd0: 0 KB (0 bits) marked out-of-sync by on > disk bit-map. > > [ 7.486017] drbd drbd0/0 drbd0: disk( Attaching -> UpToDate ) > > [ 7.486021] drbd drbd0/0 drbd0: attached to UUIDs > 32DDB2019708F68A:0000000000000000:7D97648599B446DD:7D96648599B446DD > > [ 7.486863] drbd drbd0: conn( StandAlone -> Unconnected ) > > [ 7.486871] drbd drbd0: Starting receiver thread (from drbd_w_drbd0 > [3847]) > > [ 7.486918] drbd drbd0: receiver (re)started > > [ 7.486929] drbd drbd0: conn( Unconnected -> WFConnection ) > > [ 12.340212] drbd drbd0: initial packet S crossed > > [ 22.310856] drbd drbd0: Handshake successful: Agreed network protocol > version 101 > > [ 22.311087] drbd drbd0: Feature flags enabled on protocol level: 0xf > TRIM THIN_RESYNC WRITE_SAME WRITE_ZEROES. > > [ 22.311425] drbd drbd0: conn( WFConnection -> WFReportParams ) > > [ 22.311621] drbd drbd0: Starting ack_recv thread (from drbd_r_drbd0 > [4071]) > > [ 22.400702] drbd drbd0/0 drbd0: drbd_sync_handshake: > > [ 22.400869] drbd drbd0/0 drbd0: self > 32DDB2019708F68A:0000000000000000:7D97648599B446DD:7D96648599B446DD bits:0 > flags:0 > > [ 22.401205] drbd drbd0/0 drbd0: peer > 32DDB2019708F68A:0000000000000000:7D97648599B446DC:7D96648599B446DD bits:0 > flags:0 > > [ 22.401538] drbd drbd0/0 drbd0: uuid_compare()=0 by rule 40 > > [ 22.401709] drbd drbd0/0 drbd0: peer( Unknown -> Secondary ) conn( > WFReportParams -> Connected ) pdsk( DUnknown -> UpToDate ) > > [ 22.415394] drbd drbd0/0 drbd0: role( Secondary -> Primary ) > > [ 22.506540] drbd drbd0/0 drbd0: _drbd_send_page: size=4096 len=4096 > sent=-5 > > [ 22.506773] drbd drbd0: peer( Secondary -> Unknown ) conn( Connected -> > NetworkFailure ) pdsk( UpToDate -> DUnknown ) > > [ 22.507109] drbd drbd0/0 drbd0: new current UUID > 7F8B15C04AF49C4D:32DDB2019708F68B:7D97648599B446DD:7D96648599B446DD > > [ 22.507451] drbd drbd0: ack_receiver terminated > > [ 22.507588] drbd drbd0: Terminating drbd_a_drbd0 > > [ 22.600693] drbd drbd0: Connection closed > > [ 22.600937] drbd drbd0: conn( NetworkFailure -> Unconnected ) > > [ 22.601115] drbd drbd0: receiver terminated > > [ 22.601238] drbd drbd0: Restarting receiver thread > > [ 22.601378] drbd drbd0: receiver (re)started > > [ 22.601508] drbd drbd0: conn( Unconnected -> WFConnection ) > > [ 23.260624] drbd drbd0: Handshake successful: Agreed network protocol > version 101 > > [ 23.260859] drbd drbd0: Feature flags enabled on protocol level: 0xf > TRIM THIN_RESYNC WRITE_SAME WRITE_ZEROES. > > [ 23.261187] drbd drbd0: conn( WFConnection -> WFReportParams ) > > [ 23.261367] drbd drbd0: Starting ack_recv thread (from drbd_r_drbd0 > [4071]) > > [ 23.340593] drbd drbd0/0 drbd0: drbd_sync_handshake: > > [ 23.340771] drbd drbd0/0 drbd0: self > 7F8B15C04AF49C4D:32DDB2019708F68B:7D97648599B446DD:7D96648599B446DD bits:1 > flags:0 > > [ 23.341192] drbd drbd0/0 drbd0: peer > 32DDB2019708F68A:0000000000000000:7D97648599B446DC:7D96648599B446DD bits:0 > flags:0 > > [ 23.341649] drbd drbd0/0 drbd0: uuid_compare()=1 by rule 70 > > [ 23.341824] drbd drbd0/0 drbd0: peer( Unknown -> Secondary ) conn( > WFReportParams -> WFBitMapS ) pdsk( DUnknown -> Consistent ) > > [ 23.344911] drbd drbd0/0 drbd0: send bitmap stats [Bytes(packets)]: > plain 0(0), RLE 23(1), total 23; compression: 100.0% > > [ 23.396792] drbd drbd0/0 drbd0: receive bitmap stats [Bytes(packets)]: > plain 0(0), RLE 23(1), total 23; compression: 100.0% > > [ 23.397210] drbd drbd0/0 drbd0: helper command: /sbin/drbdadm > before-resync-source minor-0 > > [ 23.407965] drbd drbd0/0 drbd0: helper command: /sbin/drbdadm > before-resync-source minor-0 exit code 0 (0x0) > > [ 23.417547] drbd drbd0/0 drbd0: conn( WFBitMapS -> SyncSource ) pdsk( > Consistent -> Inconsistent ) > > [ 23.426697] drbd drbd0/0 drbd0: Began resync as SyncSource (will sync 4 > KB [1 bits set]). > > [ 23.435638] drbd drbd0/0 drbd0: updated sync UUID > 7F8B15C04AF49C4D:32DEB2019708F68B:32DDB2019708F68B:7D97648599B446DD > > [ 23.488608] drbd drbd0/0 drbd0: _drbd_send_page: size=4096 len=4096 > sent=-5 > > [ 23.498182] drbd drbd0/0 drbd0: drbd_send_block() failed > > [ 23.508498] drbd drbd0: peer( Secondary -> Unknown ) conn( SyncSource -> > NetworkFailure ) > > [ 23.517597] drbd drbd0: ack_receiver terminated > > [ 23.527513] drbd drbd0: Terminating drbd_a_drbd0 > > [ 23.690598] drbd drbd0: Connection closed > > [ 23.701857] drbd drbd0: conn( NetworkFailure -> Unconnected ) > > [ 23.712017] drbd drbd0: receiver terminated > > [ 23.721597] drbd drbd0: Restarting receiver thread > > > > On the peer: > > > [349071.038278] drbd drbd0: conn( Unconnected -> WFConnection ) > > [349071.558245] drbd drbd0: Handshake successful: Agreed network protocol > version 101 > > [349071.562105] drbd drbd0: Feature flags enabled on protocol level: 0xf > TRIM THIN_RESYNC WRITE_SAME WRITE_ZEROES. > > [349071.569889] drbd drbd0: conn( WFConnection -> WFReportParams ) > > [349071.573802] drbd drbd0: Starting ack_recv thread (from drbd_r_drbd0 > [2660]) > > [349071.688547] drbd drbd0/0 drbd0: drbd_sync_handshake: > > [349071.692323] drbd drbd0/0 drbd0: self > 3375B2019708F68A:0000000000000000:7D97648599B446DC:7D96648599B446DD bits:1 > flags:0 > > [349071.699871] drbd drbd0/0 drbd0: peer > 7F8B15C04AF49C4D:3375B2019708F68B:3374B2019708F68B:3373B2019708F68B bits:1 > flags:0 > > [349071.707687] drbd drbd0/0 drbd0: uuid_compare()=-1 by rule 50 > > [349071.711563] drbd drbd0/0 drbd0: Becoming sync target due to disk states. > > [349071.715381] drbd drbd0/0 drbd0: peer( Unknown -> Primary ) conn( > WFReportParams -> WFBitMapT ) pdsk( DUnknown -> UpToDate ) > > [349071.723039] drbd drbd0/0 drbd0: receive bitmap stats [Bytes(packets)]: > plain 0(0), RLE 23(1), total 23; compression: 100.0% > > [349071.732489] drbd drbd0/0 drbd0: send bitmap stats [Bytes(packets)]: > plain 0(0), RLE 23(1), total 23; compression: 100.0% > > [349071.740178] drbd drbd0/0 drbd0: conn( WFBitMapT -> WFSyncUUID ) > > [349071.787113] drbd drbd0/0 drbd0: updated sync uuid > 3376B2019708F68A:0000000000000000:7D97648599B446DC:7D96648599B446DD > > [349071.794907] drbd drbd0/0 drbd0: helper command: /sbin/drbdadm > before-resync-target minor-0 > > [349071.800006] drbd drbd0/0 drbd0: helper command: /sbin/drbdadm > before-resync-target minor-0 exit code 0 (0x0) > > [349071.807737] drbd drbd0/0 drbd0: conn( WFSyncUUID -> SyncTarget ) > > [349071.811639] drbd drbd0/0 drbd0: Began resync as SyncTarget (will sync 4 > KB [1 bits set]). > > [349071.916117] drbd drbd0: sock was shut down by peer > > [349071.919955] drbd drbd0: peer( Primary -> Unknown ) conn( SyncTarget -> > BrokenPipe ) pdsk( UpToDate -> DUnknown ) > > [349071.927796] drbd drbd0: short read (expected size 4096) > > [349071.931812] drbd drbd0: error receiving RSDataReply, e: -5 l: 4096! > > [349071.935864] drbd drbd0: ack_receiver terminated > > [349071.939906] drbd drbd0: Terminating drbd_a_drbd0 > > [349072.088385] drbd drbd0: Connection closed > > [349072.092398] drbd drbd0: conn( BrokenPipe -> Unconnected ) > > [349072.096436] drbd drbd0: receiver terminated > > [349072.100469] drbd drbd0: Restarting receiver thread > > [349072.104454] drbd drbd0: receiver (re)started > > [349072.108373] drbd drbd0: conn( Unconnected -> WFConnection ) > > > -- > > Best Regards, > > Serguei

1 year, 3 months

1
0
0 0

Re: [PATCH v3] Fix srcu_struct node grpmask overflow on 64-bit systems

by Mathieu Desnoyers

On 9/4/23 08:21, Denis Arefev wrote: > The value of an arithmetic expression 1 << (cpu - sdp->mynode->grplo) > is subject to overflow due to a failure to cast operands to a larger > data type before performing arithmetic. > > The maximum result of this subtraction is defined by the RCU_FANOUT > or other srcu level-spread values assigned by rcu_init_levelspread(), > which can indeed cause the signed 32-bit integer literal ("1") to overflow > when shifted by any value greater than 31. We could expand on this: The maximum result of this subtraction is defined by the RCU_FANOUT or other srcu level-spread values assigned by rcu_init_levelspread(), which can indeed cause the signed 32-bit integer literal ("1") to overflow when shifted by any value greater than 31 on a 64-bit system. Moreover, when the subtraction value is 31, the 1 << 31 expression results in 0xffffffff80000000 when the signed integer is promoted to unsigned long on 64-bit systems due to type promotion rules, which is certainly not the intended result. > > Found by Linux Verification Center (linuxtesting.org) with SVACE. With the commit message updated with my comment above, please also add: Fixes: c7e88067c1 ("srcu: Exact tracking of srcu_data structures containing callbacks") Cc: <stable(a)vger.kernel.org> # v4.11 Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers(a)efficios.com> Thanks! Mathieu > > Signed-off-by: Denis Arefev <arefev(a)swemel.ru> > --- > v3: Changed the name of the patch, as suggested by > Mathieu Desnoyers <mathieu.desnoyers(a)efficios.com> > v2: Added fixes to the srcu_schedule_cbs_snp function as suggested by > Mathieu Desnoyers <mathieu.desnoyers(a)efficios.com> > kernel/rcu/srcutree.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/rcu/srcutree.c b/kernel/rcu/srcutree.c > index 20d7a238d675..6c18e6005ae1 100644 > --- a/kernel/rcu/srcutree.c > +++ b/kernel/rcu/srcutree.c > @@ -223,7 +223,7 @@ static bool init_srcu_struct_nodes(struct srcu_struct *ssp, gfp_t gfp_flags) > snp->grplo = cpu; > snp->grphi = cpu; > } > - sdp->grpmask = 1 << (cpu - sdp->mynode->grplo); > + sdp->grpmask = 1UL << (cpu - sdp->mynode->grplo); > } > smp_store_release(&ssp->srcu_sup->srcu_size_state, SRCU_SIZE_WAIT_BARRIER); > return true; > @@ -833,7 +833,7 @@ static void srcu_schedule_cbs_snp(struct srcu_struct *ssp, struct srcu_node *snp > int cpu; > > for (cpu = snp->grplo; cpu <= snp->grphi; cpu++) { > - if (!(mask & (1 << (cpu - snp->grplo)))) > + if (!(mask & (1UL << (cpu - snp->grplo)))) > continue; > srcu_schedule_cbs_sdp(per_cpu_ptr(ssp->sda, cpu), delay); > } -- Mathieu Desnoyers EfficiOS Inc. https://www.efficios.com

1 year, 3 months

2
6
0 0

Don't fill the kernel log with memfd_create messages

by Alex Xu (Hello71)

Hi all, Recently "memfd: improve userspace warnings for missing exec-related flags" was merged. On my system, this is a regression, not an improvement, because the entire 256k kernel log buffer (default on x86) is filled with these warnings and "__do_sys_memfd_create: 122 callbacks suppressed". I haven't investigated too closely, but the most likely cause is Wayland libraries. This is too serious of a consequence for using an old API, especially considering how recently the flags were added. The vast majority of software has not had time to add the flags: glibc does not define the macros until 2.38 which was released less than one month ago, man-pages does not document the flags, and according to Debian Code Search, only systemd, stress-ng, and strace actually pass either of these flags. Furthermore, since old kernels reject unknown flags, it's not just a matter of defining and passing the flag; every program needs to add logic to handle EINVAL and try again. Some other way needs to be found to encourage userspace to add the flags; otherwise, this message will be patched out because the kernel log becomes unusable after running unupdated programs, which will still exist even after upstreams are fixed. In particular, AppImages, flatpaks, snaps, and similar app bundles contain vendored Wayland libraries which can be difficult or impossible to update. Thanks, Alex.

1 year, 3 months

3
2
0 0

[PATCH 5.10/5.15/6.1 1/1] udf: Handle error when adding extent to a file

by Vladislav Efanov

From: Jan Kara <jack(a)suse.cz> commit 19fd80de0a8b5170ef34704c8984cca920dffa59 upstream When adding extent to a file fails, so far we've silently squelshed the error. Make sure to propagate it up properly. Signed-off-by: Jan Kara <jack(a)suse.cz> Signed-off-by: Vladislav Efanov <VEfanov(a)ispras.ru> --- Syzkaller reports this problem in 5.10 stable release. The problem has been fixed by the following patch which can be cleanly applied to the 5.10/5.15/6.1 branches. fs/udf/inode.c | 41 +++++++++++++++++++++++++++-------------- 1 file changed, 27 insertions(+), 14 deletions(-) diff --git a/fs/udf/inode.c b/fs/udf/inode.c index d114774ecdea..3e11190b7118 100644 --- a/fs/udf/inode.c +++ b/fs/udf/inode.c @@ -57,15 +57,15 @@ static int udf_update_inode(struct inode *, int); static int udf_sync_inode(struct inode *inode); static int udf_alloc_i_data(struct inode *inode, size_t size); static sector_t inode_getblk(struct inode *, sector_t, int *, int *); -static int8_t udf_insert_aext(struct inode *, struct extent_position, - struct kernel_lb_addr, uint32_t); +static int udf_insert_aext(struct inode *, struct extent_position, + struct kernel_lb_addr, uint32_t); static void udf_split_extents(struct inode *, int *, int, udf_pblk_t, struct kernel_long_ad *, int *); static void udf_prealloc_extents(struct inode *, int, int, struct kernel_long_ad *, int *); static void udf_merge_extents(struct inode *, struct kernel_long_ad *, int *); -static void udf_update_extents(struct inode *, struct kernel_long_ad *, int, - int, struct extent_position *); +static int udf_update_extents(struct inode *, struct kernel_long_ad *, int, + int, struct extent_position *); static int udf_get_block(struct inode *, sector_t, struct buffer_head *, int); static void __udf_clear_extent_cache(struct inode *inode) @@ -887,7 +887,9 @@ static sector_t inode_getblk(struct inode *inode, sector_t block, /* write back the new extents, inserting new extents if the new number * of extents is greater than the old number, and deleting extents if * the new number of extents is less than the old number */ - udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); + *err = udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); + if (*err < 0) + goto out_free; newblock = udf_get_pblock(inode->i_sb, newblocknum, iinfo->i_location.partitionReferenceNum, 0); @@ -1155,21 +1157,30 @@ static void udf_merge_extents(struct inode *inode, struct kernel_long_ad *laarr, } } -static void udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr, - int startnum, int endnum, - struct extent_position *epos) +static int udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr, + int startnum, int endnum, + struct extent_position *epos) { int start = 0, i; struct kernel_lb_addr tmploc; uint32_t tmplen; + int err; if (startnum > endnum) { for (i = 0; i < (startnum - endnum); i++) udf_delete_aext(inode, *epos); } else if (startnum < endnum) { for (i = 0; i < (endnum - startnum); i++) { - udf_insert_aext(inode, *epos, laarr[i].extLocation, - laarr[i].extLength); + err = udf_insert_aext(inode, *epos, + laarr[i].extLocation, + laarr[i].extLength); + /* + * If we fail here, we are likely corrupting the extent + * list and leaking blocks. At least stop early to + * limit the damage. + */ + if (err < 0) + return err; udf_next_aext(inode, epos, &laarr[i].extLocation, &laarr[i].extLength, 1); start++; @@ -1181,6 +1192,7 @@ static void udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr udf_write_aext(inode, epos, &laarr[i].extLocation, laarr[i].extLength, 1); } + return 0; } struct buffer_head *udf_bread(struct inode *inode, udf_pblk_t block, @@ -2215,12 +2227,13 @@ int8_t udf_current_aext(struct inode *inode, struct extent_position *epos, return etype; } -static int8_t udf_insert_aext(struct inode *inode, struct extent_position epos, - struct kernel_lb_addr neloc, uint32_t nelen) +static int udf_insert_aext(struct inode *inode, struct extent_position epos, + struct kernel_lb_addr neloc, uint32_t nelen) { struct kernel_lb_addr oeloc; uint32_t oelen; int8_t etype; + int err; if (epos.bh) get_bh(epos.bh); @@ -2230,10 +2243,10 @@ static int8_t udf_insert_aext(struct inode *inode, struct extent_position epos, neloc = oeloc; nelen = (etype << 30) | oelen; } - udf_add_aext(inode, &epos, &neloc, nelen, 1); + err = udf_add_aext(inode, &epos, &neloc, nelen, 1); brelse(epos.bh); - return (nelen >> 30); + return err; } int8_t udf_delete_aext(struct inode *inode, struct extent_position epos) -- 2.34.1

1 year, 3 months

1
0
0 0

Re: [PATCH 6.5 00/34] 6.5.2-rc1 review

by Ronald Warsow

Hi Greg 6.5.2-rc1 compiles, boots and runs here on x86_64 (Intel Rocket Lake, i5-11400) Thanks Tested-by: Ronald Warsow <rwarsow(a)gmx.de>

1 year, 3 months

1
0
0 0

[PATCH] quota: explicitly forbid quota files from being encrypted

by Eric Biggers

From: Eric Biggers <ebiggers(a)google.com> Since commit d7e7b9af104c ("fscrypt: stop using keyrings subsystem for fscrypt_master_key"), xfstest generic/270 causes a WARNING when run on f2fs with test_dummy_encryption in the mount options: $ kvm-xfstests -c f2fs/encrypt generic/270 [...] WARNING: CPU: 1 PID: 2453 at fs/crypto/keyring.c:240 fscrypt_destroy_keyring+0x1f5/0x260 The cause of the WARNING is that not all encrypted inodes have been evicted before fscrypt_destroy_keyring() is called, which violates an assumption. This happens because the test uses an external quota file, which gets automatically encrypted due to test_dummy_encryption. Encryption of quota files has never really been supported. On ext4, ext4_quota_read() does not decrypt the data, so encrypted quota files are always considered invalid on ext4. On f2fs, f2fs_quota_read() uses the pagecache, so trying to use an encrypted quota file gets farther, resulting in the issue described above being possible. But this was never intended to be possible, and there is no use case for it. Therefore, make the quota support layer explicitly reject using IS_ENCRYPTED inodes when quotaon is attempted. Cc: stable(a)vger.kernel.org Signed-off-by: Eric Biggers <ebiggers(a)google.com> --- fs/quota/dquot.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index 9e72bfe8bbad9..7e268cd2727cc 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c @@ -2339,6 +2339,20 @@ static int vfs_setup_quota_inode(struct inode *inode, int type) if (sb_has_quota_loaded(sb, type)) return -EBUSY; + /* + * Quota files should never be encrypted. They should be thought of as + * filesystem metadata, not user data. New-style internal quota files + * cannot be encrypted by users anyway, but old-style external quota + * files could potentially be incorrectly created in an encrypted + * directory, hence this explicit check. Some reasons why encrypted + * quota files don't work include: (1) some filesystems that support + * encryption don't handle it in their quota_read and quota_write, and + * (2) cleaning up encrypted quota files at unmount would need special + * consideration, as quota files are cleaned up later than user files. + */ + if (IS_ENCRYPTED(inode)) + return -EINVAL; + dqopt->files[type] = igrab(inode); if (!dqopt->files[type]) return -EIO; base-commit: 708283abf896dd4853e673cc8cba70acaf9bf4ea -- 2.42.0

1 year, 3 months

2
1
0 0

Betrugsopfer

by (IMF) SCAM VICTIMS

Sehr geehrter E-Mail-Besitzer, Der Internationale Währungsfonds (IWF) entschädigt alle Betrugsopfer und Ihre E-Mail-Adresse wurde auf der Liste der Betrugsopfer gefunden. Dieses Western Union-Büro wurde vom IWF beauftragt Ihnen Ihre Vergütung per Western Union Money Transfer zu überweisen. Wir haben uns jedoch entschieden Ihre eigene Zahlung über Geldtransfer der Westunion in Höhe von €5,000, pro Tag vorzunehmen bis die Gesamtsumme von €1,500.000.00, vollständig an Sie überwiesen wurde. Wir können die Zahlung möglicherweise nicht nur mit Ihrer E-Mail-Adresse senden daher benötigen wir Ihre Informationen darüber wohin wir das Geld an Sie senden wie z. B.: Name des Adressaten ________________ Adresse________________ Land__________________ Telefonnummer________________ Angehängte Kopie Ihres Ausweises______________ Das Alter ________________________ Wir beginnen mit der Übertragung sobald wir Ihre Informationen erhalten haben: Kontakt E-Mail: ( wuwumoneytransfer5000(a)hotmail.com) Getreu, Herr Anthony Duru, Direktor von Geldtransfer der Westunion

1 year, 3 months

1
0
0 0

Efektivní sledování a optimalizace výroby pro vaši společnost

by Michal Rmoutil

Dobré ráno, máte možnost sledovat stav každého stroje a výrobního procesu z kanceláře, konferenční místnosti nebo dokonce z domova či na cestách – na vašem telefonu? Poskytujeme rychle implementovatelný a snadno použitelný nástroj, který zachytí i několikasekundový mikroprostoj a okamžitě přepočítá využití stroje v kontextu dané výrobní zakázky. Kdykoli vidíte stav objednávky a jste informováni o případném snížení efektivity. Systém sám analyzuje data a připravuje cenné reporty, což operátorům umožňuje soustředit se na výrobní cíl. Cíl je jednoduchý: jeden pohled – celá továrna. Čekám na odpověď, jestli vidíte možnost využití takového nástroje ve vaší firmě. Pozdravy Michal Rmoutil

1 year, 3 months

1
0
0 0

[PATCH 5.10 1/1] udf: Handle error when adding extent to a file

by Vladislav Efanov

From: Jan Kara <jack(a)suse.cz> commit 19fd80de0a8b5170ef34704c8984cca920dffa59 upstream When adding extent to a file fails, so far we've silently squelshed the error. Make sure to propagate it up properly. Signed-off-by: Jan Kara <jack(a)suse.cz> Signed-off-by: Vladislav Efanov <VEfanov(a)ispras.ru> --- Syzkaller reports this problem in 5.10 stable release. The problem has been fixed by the following patch which can be cleanly applied to the 5.10 branch. fs/udf/inode.c | 41 +++++++++++++++++++++++++++-------------- 1 file changed, 27 insertions(+), 14 deletions(-) diff --git a/fs/udf/inode.c b/fs/udf/inode.c index d114774ecdea..3e11190b7118 100644 --- a/fs/udf/inode.c +++ b/fs/udf/inode.c @@ -57,15 +57,15 @@ static int udf_update_inode(struct inode *, int); static int udf_sync_inode(struct inode *inode); static int udf_alloc_i_data(struct inode *inode, size_t size); static sector_t inode_getblk(struct inode *, sector_t, int *, int *); -static int8_t udf_insert_aext(struct inode *, struct extent_position, - struct kernel_lb_addr, uint32_t); +static int udf_insert_aext(struct inode *, struct extent_position, + struct kernel_lb_addr, uint32_t); static void udf_split_extents(struct inode *, int *, int, udf_pblk_t, struct kernel_long_ad *, int *); static void udf_prealloc_extents(struct inode *, int, int, struct kernel_long_ad *, int *); static void udf_merge_extents(struct inode *, struct kernel_long_ad *, int *); -static void udf_update_extents(struct inode *, struct kernel_long_ad *, int, - int, struct extent_position *); +static int udf_update_extents(struct inode *, struct kernel_long_ad *, int, + int, struct extent_position *); static int udf_get_block(struct inode *, sector_t, struct buffer_head *, int); static void __udf_clear_extent_cache(struct inode *inode) @@ -887,7 +887,9 @@ static sector_t inode_getblk(struct inode *inode, sector_t block, /* write back the new extents, inserting new extents if the new number * of extents is greater than the old number, and deleting extents if * the new number of extents is less than the old number */ - udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); + *err = udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); + if (*err < 0) + goto out_free; newblock = udf_get_pblock(inode->i_sb, newblocknum, iinfo->i_location.partitionReferenceNum, 0); @@ -1155,21 +1157,30 @@ static void udf_merge_extents(struct inode *inode, struct kernel_long_ad *laarr, } } -static void udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr, - int startnum, int endnum, - struct extent_position *epos) +static int udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr, + int startnum, int endnum, + struct extent_position *epos) { int start = 0, i; struct kernel_lb_addr tmploc; uint32_t tmplen; + int err; if (startnum > endnum) { for (i = 0; i < (startnum - endnum); i++) udf_delete_aext(inode, *epos); } else if (startnum < endnum) { for (i = 0; i < (endnum - startnum); i++) { - udf_insert_aext(inode, *epos, laarr[i].extLocation, - laarr[i].extLength); + err = udf_insert_aext(inode, *epos, + laarr[i].extLocation, + laarr[i].extLength); + /* + * If we fail here, we are likely corrupting the extent + * list and leaking blocks. At least stop early to + * limit the damage. + */ + if (err < 0) + return err; udf_next_aext(inode, epos, &laarr[i].extLocation, &laarr[i].extLength, 1); start++; @@ -1181,6 +1192,7 @@ static void udf_update_extents(struct inode *inode, struct kernel_long_ad *laarr udf_write_aext(inode, epos, &laarr[i].extLocation, laarr[i].extLength, 1); } + return 0; } struct buffer_head *udf_bread(struct inode *inode, udf_pblk_t block, @@ -2215,12 +2227,13 @@ int8_t udf_current_aext(struct inode *inode, struct extent_position *epos, return etype; } -static int8_t udf_insert_aext(struct inode *inode, struct extent_position epos, - struct kernel_lb_addr neloc, uint32_t nelen) +static int udf_insert_aext(struct inode *inode, struct extent_position epos, + struct kernel_lb_addr neloc, uint32_t nelen) { struct kernel_lb_addr oeloc; uint32_t oelen; int8_t etype; + int err; if (epos.bh) get_bh(epos.bh); @@ -2230,10 +2243,10 @@ static int8_t udf_insert_aext(struct inode *inode, struct extent_position epos, neloc = oeloc; nelen = (etype << 30) | oelen; } - udf_add_aext(inode, &epos, &neloc, nelen, 1); + err = udf_add_aext(inode, &epos, &neloc, nelen, 1); brelse(epos.bh); - return (nelen >> 30); + return err; } int8_t udf_delete_aext(struct inode *inode, struct extent_position epos) -- 2.34.1

1 year, 3 months

3
5
0 0

2024

2023

2022

2021

2020

2019

2018

2017

Linux-stable-mirror September 2023