April 2025 - Devel - Nfs Ganesha List Archives

[S] Change in ...nfs-ganesha[next]: Allow max groups membership to be configurable

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1214084?usp=email to review the following change. Change subject: Allow max groups membership to be configurable ...................................................................... Allow max groups membership to be configurable This change adds a new configuration item to set a maximum number of groups a user can be a part of. The default value is 2000. The parameter name is Max_Groups_Membership. Change-Id: Id74207d9aa985dac59ad8236c2872ae074e43f43 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/support/nfs_read_conf.c M src/support/uid2grp.c 4 files changed, 23 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/84/1214084/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1214084?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Id74207d9aa985dac59ad8236c2872ae074e43f43 Gerrit-Change-Number: 1214084 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 month

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: idmapper: should not set idmapping_enabled to true by default

by vicente (GerritHub)

vicente has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1214048?usp=email ) Change subject: idmapper: should not set idmapping_enabled to true by default ...................................................................... idmapper: should not set idmapping_enabled to true by default We remove the one calling path for idmapper_set_owner_domain() on commit 69da0b4. But in some cases, we would not be able to call the idmapper_set_owner_domain(), which would cause the owner_domain not to be initialized. For example, the current idmapping_enabled default is true. We will pass the nfs_param.directory_services_param.idmapping_active as status_enabled. If the idmapping_active is also true, we will skip the first time init to calling idmapper_set_owner_domain(). To resolve this, we should set idmapping_enabled as false by default. That will ensure we can execute the idmapper_set_owner_domain() for initialization. Change-Id: I1a3d6e88a59d3943823892f72ee58265f9a9f156 Signed-off-by: Vicente Cheng <vicente.cheng(a)suse.com> --- M src/idmapper/idmapper.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/48/1214048/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1214048?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I1a3d6e88a59d3943823892f72ee58265f9a9f156 Gerrit-Change-Number: 1214048 Gerrit-PatchSet: 1 Gerrit-Owner: vicente <freeze.bilsted(a)gmail.com>

1 month

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: Metrics: Added filepath label for export_id metrics

by Name of user not set (GerritHub)

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213981?usp=email ) Change subject: Metrics: Added filepath label for export_id metrics ...................................................................... Metrics: Added filepath label for export_id metrics Change-Id: I80be2557ec23bd06f9fc92edb46e907dce57868b Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> Signed-off-by: Sreedhat Agraharam <sragraha(a)redhat.com> --- M src/MainNFSD/nfs_metrics.c M src/include/nfs_metrics.h M src/monitoring/dynamic_metrics.cc M src/monitoring/include/dynamic_metrics.h M src/support/server_stats.c 5 files changed, 34 insertions(+), 39 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/1213981/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213981?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I80be2557ec23bd06f9fc92edb46e907dce57868b Gerrit-Change-Number: 1213981 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 month, 1 week

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Coverity Issue - 542648

by Name of user not set (GerritHub)

raprasad(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213847?usp=email ) Change subject: Coverity Issue - 542648 ...................................................................... Coverity Issue - 542648 Fix to avoid the INTEGER_OVERFLOW issue. The variable new_count is a signed integer, which can potentially cause an INTEGER_OVERFLOW problem when used with gsh_realloc() or while updating an unsigned integer variable (acldata.naces) with its value. The function posix_acl_2_fsal_acl() may return -1, which gets assigned to both new_count and new_i_count. This scenario can lead to an INTEGER_OVERFLOW issue. Change-Id: I7a4e5bec06776b9c443c6851caaedf8b1a8e61a4 Signed-off-by: Rajesh Prasad <raprasad(a)redhat.com> --- M src/FSAL/FSAL_CEPH/internal.c 1 file changed, 6 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/1213847/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213847?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I7a4e5bec06776b9c443c6851caaedf8b1a8e61a4 Gerrit-Change-Number: 1213847 Gerrit-PatchSet: 1 Gerrit-Owner: raprasad(a)redhat.com

1 month, 1 week

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: Config: Add support for Monitoring_Addr

by Sachin Punadikar (GerritHub)

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213832?usp=email ) Change subject: Config: Add support for Monitoring_Addr ...................................................................... Config: Add support for Monitoring_Addr Provision to listen on specific IP address for Ganesha monitoring module. Change-Id: I2b5a20a58cbe04a12017fa4d668daaacf403df45 Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/MainNFSD/nfs_main.c M src/config_samples/config.txt M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/monitoring/include/prometheus_exposer.h M src/monitoring/prometheus_exposer.cc M src/support/nfs_read_conf.c 7 files changed, 47 insertions(+), 17 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/32/1213832/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213832?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I2b5a20a58cbe04a12017fa4d668daaacf403df45 Gerrit-Change-Number: 1213832 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 month, 1 week

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Coverity 502077 Overflowed array index write: In cidr_net_supernet: W...

by Suhas Athani (GerritHub)

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213830?usp=email ) Change subject: Coverity 502077 Overflowed array index write: In cidr_net_supernet: Write to array at integer-overflowed index. Fix integer underflow risk by validating CIDR prefix length. ...................................................................... Coverity 502077 Overflowed array index write: In cidr_net_supernet: Write to array at integer-overflowed index. Fix integer underflow risk by validating CIDR prefix length. Change-Id: Icd31e18d4f830f3b6ba4816fc5a39761df21b239 Signed-off-by: s-athani <suhas.athani(a)ibm.com> --- M src/cidr/cidr_net.c 1 file changed, 6 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/30/1213830/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213830?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Icd31e18d4f830f3b6ba4816fc5a39761df21b239 Gerrit-Change-Number: 1213830 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 month, 1 week

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Coverty check: Update pthread open syscalls with internal macro

by Name of user not set (GerritHub)

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213781?usp=email ) Change subject: Coverty check: Update pthread open syscalls with internal macro ...................................................................... Coverty check: Update pthread open syscalls with internal macro Change-Id: I95c34b58516f02c0c490aa58ac33c941fafebdea Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> Signed-off-by: Sreedhat Agraharam <sragraha(a)redhat.com> --- M src/MainNFSD/nfs_rpc_callback.c M src/SAL/nfs4_clientid.c M src/SAL/nfs4_recovery.c 3 files changed, 17 insertions(+), 17 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/1213781/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213781?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I95c34b58516f02c0c490aa58ac33c941fafebdea Gerrit-Change-Number: 1213781 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 month, 1 week

1
0
0 / 0

Announce Push of V7-dev.12

by Frank Filz

Branch next Tag:V7-dev.12 Merge Highlights * several Coverity fixes * CEPH: Reclaim support for multiple ceph clients Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: fc1daa6a6 Frank S. Filz V7-dev.12 b9547682d Sreedhat Agraharam CID 542662 : Intialized scalar variable of ceph_statx 4310b363d VidyaThumukunta Coverity 72124: Fix for file descriptor leak in nlm_send_async 4016b627f VidyaThumukunta Coverity 542661: Fix constant expression in grace status condition logic 87c7c8af3 s-athani Coverity 542657: Calling display_client_id_rec dereferences freed pointer clientid->cid_client_record. Fix Added. c8c6b8d8c Sachin Punadikar Coverity fix : Fix memory leak (CID 542667) 309d963c0 Sachin Punadikar CEPH: Reclaim support for multiple ceph clients 78bdb3696 Frank S. Filz Checkpatch - stop comnplaining about indented labels - that flags C++ code

1 month, 1 week

1
0
0 / 0

The prime porn directory website brings you the best websites with diverse and very delicious porn videos to satisfy you. The porn directory is very extensive for you, but you will find some impressive videos full of uncontrolled sexual action. The top porn brings rough porn where the submissive does everything the dominator or owner asks, do not miss these porn videos.

by pawewo1417＠cxnlab.com

The prime porn directory website brings you the best websites with diverse and very delicious porn videos to satisfy you. The porn directory is very extensive for you, but you will find some impressive videos full of uncontrolled sexual action. The top porn brings rough porn where the submissive does everything the dominator or owner asks, do not miss these porn videos. https://www.adultfunlist.com

1 month, 1 week

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Coverity 542661: Fix constant expression in grace status condition logic

by Name of user not set (GerritHub)

Vidya.Thumukunta(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213682?usp=email ) Change subject: Coverity 542661: Fix constant expression in grace status condition logic ...................................................................... Coverity 542661: Fix constant expression in grace status condition logic Change-Id: I3defc1b29e002fa92c7c4fca7c874b11272d1139 Signed-off-by: VidyaThumukunta <Vidya.Thumukunta(a)ibm.com> --- M src/SAL/nfs4_recovery.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/82/1213682/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213682?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I3defc1b29e002fa92c7c4fca7c874b11272d1139 Gerrit-Change-Number: 1213682 Gerrit-PatchSet: 1 Gerrit-Owner: Vidya.Thumukunta(a)ibm.com

1 month, 2 weeks

1
0
0 / 0

2025

2024

2023

2022

2021

2020

2019

2018

Devel April 2025