* [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting
@ 2015-04-16 12:51 Kinglong Mee
2015-04-17 18:15 ` Scott Mayhew
2015-05-07 14:17 ` Steve Dickson
0 siblings, 2 replies; 4+ messages in thread
From: Kinglong Mee @ 2015-04-16 12:51 UTC (permalink / raw)
To: Steve Dickson; +Cc: linux-nfs@vger.kernel.org, smayhew, kinglongmee
NFSD return reply of client's SECINFO_NO_NAME request with four
security information number as, 1 1 0 0
It's caused by commit 4a1ad4aa30,
"mountd: Enable all auth flavors on pseudofs exports"
This patch removes duplicate secinfo and invalid secinfo (zero).
Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
---
support/nfs/exports.c | 2 +-
utils/mountd/v4root.c | 3 +++
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/support/nfs/exports.c b/support/nfs/exports.c
index 761a046..0aea6f1 100644
--- a/support/nfs/exports.c
+++ b/support/nfs/exports.c
@@ -408,7 +408,7 @@ int secinfo_addflavor(struct flav_info *flav, struct exportent *ep)
struct sec_entry *p;
for (p=ep->e_secinfo; p->flav; p++) {
- if (p->flav == flav)
+ if (p->flav == flav || p->flav->fnum == flav->fnum)
return p - ep->e_secinfo;
}
if (p - ep->e_secinfo >= SECFLAVOR_COUNT) {
diff --git a/utils/mountd/v4root.c b/utils/mountd/v4root.c
index 429ebb8..d521725 100644
--- a/utils/mountd/v4root.c
+++ b/utils/mountd/v4root.c
@@ -69,6 +69,9 @@ set_pseudofs_security(struct exportent *pseudo, int flags)
for (flav = flav_map; flav < flav_map + flav_map_size; flav++) {
struct sec_entry *new;
+ if (!flav->fnum)
+ continue;
+
i = secinfo_addflavor(flav, pseudo);
new = &pseudo->e_secinfo[i];
--
2.3.5
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting
2015-04-16 12:51 [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting Kinglong Mee
@ 2015-04-17 18:15 ` Scott Mayhew
2015-05-06 13:38 ` Kinglong Mee
2015-05-07 14:17 ` Steve Dickson
1 sibling, 1 reply; 4+ messages in thread
From: Scott Mayhew @ 2015-04-17 18:15 UTC (permalink / raw)
To: Kinglong Mee; +Cc: Steve Dickson, linux-nfs@vger.kernel.org
On Thu, 16 Apr 2015, Kinglong Mee wrote:
> NFSD return reply of client's SECINFO_NO_NAME request with four
> security information number as, 1 1 0 0
>
> It's caused by commit 4a1ad4aa30,
> "mountd: Enable all auth flavors on pseudofs exports"
>
> This patch removes duplicate secinfo and invalid secinfo (zero).
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
This is because flav_map has both AUTH_UNIX/AUTH_SYS and
AUTH_NULL/AUTH_NONE. I had discussed it with Steve, but since I didn't
observe any problems caused by it in my testing I just left them in.
Acked-by: Scott Mayhew <smayhew@redhat.com>
> ---
> support/nfs/exports.c | 2 +-
> utils/mountd/v4root.c | 3 +++
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/support/nfs/exports.c b/support/nfs/exports.c
> index 761a046..0aea6f1 100644
> --- a/support/nfs/exports.c
> +++ b/support/nfs/exports.c
> @@ -408,7 +408,7 @@ int secinfo_addflavor(struct flav_info *flav, struct exportent *ep)
> struct sec_entry *p;
>
> for (p=ep->e_secinfo; p->flav; p++) {
> - if (p->flav == flav)
> + if (p->flav == flav || p->flav->fnum == flav->fnum)
> return p - ep->e_secinfo;
> }
> if (p - ep->e_secinfo >= SECFLAVOR_COUNT) {
> diff --git a/utils/mountd/v4root.c b/utils/mountd/v4root.c
> index 429ebb8..d521725 100644
> --- a/utils/mountd/v4root.c
> +++ b/utils/mountd/v4root.c
> @@ -69,6 +69,9 @@ set_pseudofs_security(struct exportent *pseudo, int flags)
> for (flav = flav_map; flav < flav_map + flav_map_size; flav++) {
> struct sec_entry *new;
>
> + if (!flav->fnum)
> + continue;
> +
> i = secinfo_addflavor(flav, pseudo);
> new = &pseudo->e_secinfo[i];
>
> --
> 2.3.5
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting
2015-04-17 18:15 ` Scott Mayhew
@ 2015-05-06 13:38 ` Kinglong Mee
0 siblings, 0 replies; 4+ messages in thread
From: Kinglong Mee @ 2015-05-06 13:38 UTC (permalink / raw)
To: Steve Dickson; +Cc: Scott Mayhew, linux-nfs@vger.kernel.org
On 4/18/2015 2:15 AM, Scott Mayhew wrote:
> On Thu, 16 Apr 2015, Kinglong Mee wrote:
>
>> NFSD return reply of client's SECINFO_NO_NAME request with four
>> security information number as, 1 1 0 0
>>
>> It's caused by commit 4a1ad4aa30,
>> "mountd: Enable all auth flavors on pseudofs exports"
>>
>> This patch removes duplicate secinfo and invalid secinfo (zero).
>>
>> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
>
> This is because flav_map has both AUTH_UNIX/AUTH_SYS and
> AUTH_NULL/AUTH_NONE. I had discussed it with Steve, but since I didn't
> observe any problems caused by it in my testing I just left them in.
>
> Acked-by: Scott Mayhew <smayhew@redhat.com>
Also, this one?
thanks,
Kinglong Mee
>
>> ---
>> support/nfs/exports.c | 2 +-
>> utils/mountd/v4root.c | 3 +++
>> 2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/support/nfs/exports.c b/support/nfs/exports.c
>> index 761a046..0aea6f1 100644
>> --- a/support/nfs/exports.c
>> +++ b/support/nfs/exports.c
>> @@ -408,7 +408,7 @@ int secinfo_addflavor(struct flav_info *flav, struct exportent *ep)
>> struct sec_entry *p;
>>
>> for (p=ep->e_secinfo; p->flav; p++) {
>> - if (p->flav == flav)
>> + if (p->flav == flav || p->flav->fnum == flav->fnum)
>> return p - ep->e_secinfo;
>> }
>> if (p - ep->e_secinfo >= SECFLAVOR_COUNT) {
>> diff --git a/utils/mountd/v4root.c b/utils/mountd/v4root.c
>> index 429ebb8..d521725 100644
>> --- a/utils/mountd/v4root.c
>> +++ b/utils/mountd/v4root.c
>> @@ -69,6 +69,9 @@ set_pseudofs_security(struct exportent *pseudo, int flags)
>> for (flav = flav_map; flav < flav_map + flav_map_size; flav++) {
>> struct sec_entry *new;
>>
>> + if (!flav->fnum)
>> + continue;
>> +
>> i = secinfo_addflavor(flav, pseudo);
>> new = &pseudo->e_secinfo[i];
>>
>> --
>> 2.3.5
>>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting
2015-04-16 12:51 [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting Kinglong Mee
2015-04-17 18:15 ` Scott Mayhew
@ 2015-05-07 14:17 ` Steve Dickson
1 sibling, 0 replies; 4+ messages in thread
From: Steve Dickson @ 2015-05-07 14:17 UTC (permalink / raw)
To: Kinglong Mee; +Cc: linux-nfs@vger.kernel.org, smayhew
On 04/16/2015 08:51 AM, Kinglong Mee wrote:
> NFSD return reply of client's SECINFO_NO_NAME request with four
> security information number as, 1 1 0 0
>
> It's caused by commit 4a1ad4aa30,
> "mountd: Enable all auth flavors on pseudofs exports"
>
> This patch removes duplicate secinfo and invalid secinfo (zero).
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
> ---
> support/nfs/exports.c | 2 +-
> utils/mountd/v4root.c | 3 +++
> 2 files changed, 4 insertions(+), 1 deletion(-)
Committed...
steved.
>
> diff --git a/support/nfs/exports.c b/support/nfs/exports.c
> index 761a046..0aea6f1 100644
> --- a/support/nfs/exports.c
> +++ b/support/nfs/exports.c
> @@ -408,7 +408,7 @@ int secinfo_addflavor(struct flav_info *flav, struct exportent *ep)
> struct sec_entry *p;
>
> for (p=ep->e_secinfo; p->flav; p++) {
> - if (p->flav == flav)
> + if (p->flav == flav || p->flav->fnum == flav->fnum)
> return p - ep->e_secinfo;
> }
> if (p - ep->e_secinfo >= SECFLAVOR_COUNT) {
> diff --git a/utils/mountd/v4root.c b/utils/mountd/v4root.c
> index 429ebb8..d521725 100644
> --- a/utils/mountd/v4root.c
> +++ b/utils/mountd/v4root.c
> @@ -69,6 +69,9 @@ set_pseudofs_security(struct exportent *pseudo, int flags)
> for (flav = flav_map; flav < flav_map + flav_map_size; flav++) {
> struct sec_entry *new;
>
> + if (!flav->fnum)
> + continue;
> +
> i = secinfo_addflavor(flav, pseudo);
> new = &pseudo->e_secinfo[i];
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-05-07 14:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-16 12:51 [PATCH] rpc.mountd: Remove duplicate and invalid secinfo setting Kinglong Mee
2015-04-17 18:15 ` Scott Mayhew
2015-05-06 13:38 ` Kinglong Mee
2015-05-07 14:17 ` Steve Dickson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).