Project

General

Profile

Actions

Bug #12294

closed

rfs3_readdir()/rfs3_readdirplus(): Duplicate vattr_to_post_op_attr() call

Added by Marcel Telka over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
nfs - NFS server and client
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:

Description

There are duplicate vattr_to_post_op_attr() calls at the end of rfs3_readdir() and rfs3_readdirplus(). The second one could be safely removed.


Related issues

Related to illumos gate - Feature #11083: support NFS server in zoneClosedDan McDonald

Actions
Actions #1

Updated by Marcel Telka over 1 year ago

Actions #2

Updated by Marcel Telka over 1 year ago

  • Priority changed from Normal to Low
Actions #4

Updated by Marcel Telka over 1 year ago

  • Status changed from In Progress to Pending RTI
Actions #5

Updated by Dan McDonald over 1 year ago

NOTE that "at the end" can be stated more specifically as "in sections that are essentially basic-blocks", namely the "out1:" goto labels of these two functions. There's no looping or other trickery than can make this happen. And the input parameter "vap" doesn't change over the course of this block, so the two calls are idempotent.

These duplicates were introduced by accident from the nfs-zone work (#11083).

Actions #6

Updated by Electric Monk over 1 year ago

  • Status changed from Pending RTI to Closed
  • % Done changed from 0 to 100

git commit d5cb55e4df73e14ab5a6a4266b32230c8e73e54e

commit  d5cb55e4df73e14ab5a6a4266b32230c8e73e54e
Author: Marcel Telka <marcel@telka.sk>
Date:   2020-02-11T14:40:12.000Z

    12294 rfs3_readdir()/rfs3_readdirplus(): Duplicate vattr_to_post_op_attr() call
    Reviewed by: Vitaliy Gusev <gusev.vitaliy@icloud.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Actions

Also available in: Atom PDF