Changeset 9c4cf0d in mainline for uspace/app


Ignore:
Timestamp:
2017-04-02T11:24:06Z (9 years ago)
Author:
Jakub Jermar <jakub@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
151f1cc
Parents:
b19e892
Message:

Rename close() to vfs_put()

This is motivated mainly by the fact that a file handle does not
necessarily correspond to an open file and close() was no longer the
the opposite operation to open().

Location:
uspace/app
Files:
17 edited

Legend:

Unmodified
Added
Removed

  • uspace/app/bdsh/cmds/modules/cat/cat.c

    rb19e892 r9c4cf0d  
    203203
    204204        if (NULL == (buff = (char *) malloc(blen + 1))) {
    205                 close(fd);
     205                vfs_put(fd);
    206206                printf("Unable to allocate enough memory to read %s\n",
    207207                    fname);
    … …  
    213213
    214214                if (vfs_stat(fd, &st) != EOK) {
    215                         close(fd);
     215                        vfs_put(fd);
    216216                        free(buff);
    217217                        printf("Unable to vfs_stat %d\n", fd);
    … …  
    287287        } while (bytes > 0 && !should_quit && (count < length || length == CAT_FULL_FILE));
    288288
    289         close(fd);
     289        vfs_put(fd);
    290290        if (bytes == -1) {
    291291                printf("Error reading %s\n", fname);

  • uspace/app/bdsh/cmds/modules/cmp/cmp.c

    rb19e892 r9c4cf0d  
    116116end:
    117117        if (fd[0] >= 0)
    118                 close(fd[0]);
     118                vfs_put(fd[0]);
    119119        if (fd[1] >= 0)
    120                 close(fd[1]);
     120                vfs_put(fd[1]);
    121121        return rc;
    122122}

  • uspace/app/bdsh/cmds/modules/cp/cp.c

    rb19e892 r9c4cf0d  
    391391        if (fd2 < 0) {
    392392                printf("Unable to open destination file %s\n", dest);
    393                 close(fd1);
     393                vfs_put(fd1);
    394394                return -1;
    395395        }
    … …  
    397397        if (vfs_stat(fd1, &st) != EOK) {
    398398                printf("Unable to fstat %d\n", fd1);
    399                 close(fd1);
    400                 close(fd2);
     399                vfs_put(fd1);
     400                vfs_put(fd2);
    401401                return -1;     
    402402        }
    … …  
    425425
    426426out:
    427         close(fd1);
    428         close(fd2);
     427        vfs_put(fd1);
     428        vfs_put(fd2);
    429429        if (buff)
    430430                free(buff);

  • uspace/app/bdsh/cmds/modules/mkfile/mkfile.c

    rb19e892 r9c4cf0d  
    168168                rc2 = write(fd, &pos, &byte, sizeof(char));
    169169                if (rc2 < 0) {
    170                         close(fd);
     170                        vfs_put(fd);
    171171                        goto error;
    172172                }
    … …  
    186186                if (rc <= 0) {
    187187                        printf("%s: Error writing file (%d).\n", cmdname, errno);
    188                         close(fd);
     188                        vfs_put(fd);
    189189                        free(buffer);
    190190                        return CMD_FAILURE;
    … …  
    195195        free(buffer);
    196196
    197         if (close(fd) < 0)
     197        if (vfs_put(fd) < 0)
    198198                goto error;
    199199

  • uspace/app/bdsh/cmds/modules/rm/rm.c

    rb19e892 r9c4cf0d  
    152152        fd = vfs_lookup(path, WALK_REGULAR);
    153153        if (fd >= 0) {
    154                 close(fd);
     154                vfs_put(fd);
    155155                return RM_FILE;
    156156        }

  • uspace/app/bdsh/cmds/modules/touch/touch.c

    rb19e892 r9c4cf0d  
    133133                        continue;
    134134                } else {
    135                         close(fd);
     135                        vfs_put(fd);
    136136                        fd = -1;
    137137                }

  • uspace/app/bdsh/exec.c

    rb19e892 r9c4cf0d  
    6161        fd = vfs_lookup_open(f, WALK_REGULAR, MODE_READ);
    6262        if (fd >= 0) {
    63                 close(fd);
     63                vfs_put(fd);
    6464                return 0;
    6565        } else

  • uspace/app/getterm/getterm.c

    rb19e892 r9c4cf0d  
    7474                        return;
    7575               
    76                 if (close(oldfd))
     76                if (vfs_put(oldfd))
    7777                        return;
    7878        }

  • uspace/app/redir/redir.c

    rb19e892 r9c4cf0d  
    6969                        return;
    7070               
    71                 if (close(oldfd))
     71                if (vfs_put(oldfd))
    7272                        return;
    7373        }

  • uspace/app/sysinst/futil.c

    rb19e892 r9c4cf0d  
    8383        } while (true);
    8484
    85         (void) close(sf);
     85        (void) vfs_put(sf);
    8686
    87         rc = close(df);
     87        rc = vfs_put(df);
    8888        if (rc < 0)
    8989                return EIO;
    … …  
    166166
    167167        if (vfs_stat(sf, &st) != EOK) {
    168                 close(sf);
     168                vfs_put(sf);
    169169                return EIO;
    170170        }       
    … …  
    174174        data = calloc(fsize, 1);
    175175        if (data == NULL) {
    176                 close(sf);
     176                vfs_put(sf);
    177177                return ENOMEM;
    178178        }
    … …  
    180180        nr = read(sf, (aoff64_t []) { 0 }, data, fsize);
    181181        if (nr != (ssize_t)fsize) {
    182                 close(sf);
     182                vfs_put(sf);
    183183                free(data);
    184184                return EIO;
    185185        }
    186186
    187         (void) close(sf);
     187        (void) vfs_put(sf);
    188188        *rdata = data;
    189189        *rsize = fsize;

  • uspace/app/taskdump/symtab.c

    rb19e892 r9c4cf0d  
    165165
    166166        free(shstrt);
    167         close(fd);
     167        vfs_put(fd);
    168168
    169169        if (stab->sym == NULL || stab->strtab == NULL) {

  • uspace/app/tester/hw/misc/virtchar1.c

    rb19e892 r9c4cf0d  
    6969        async_sess_t *sess = vfs_fd_session(fd, INTERFACE_DDF);
    7070        if (!sess) {
    71                 close(fd);
     71                vfs_put(fd);
    7272                TPRINTF("   ...error: %s\n", str_error(errno));
    7373                return "Failed to get session to device";
    … …  
    9090        TPRINTF(" Closing session and file descriptor\n");
    9191        async_hangup(sess);
    92         close(fd);
     92        vfs_put(fd);
    9393       
    9494        return NULL;

  • uspace/app/tester/mm/pager1.c

    rb19e892 r9c4cf0d  
    5454
    5555        if (write(fd, (aoff64_t []) {0}, text, sizeof(text)) != sizeof(text)) {
    56                 close(fd);
     56                vfs_put(fd);
    5757                return NULL;
    5858        }
    … …  
    6565
    6666        if (!vfs_pager_sess) {
    67                 close(fd);
     67                vfs_put(fd);
    6868                return NULL;
    6969        }
    … …  
    7474            AS_AREA_READ | AS_AREA_CACHEABLE, vfs_pager_sess, fd, 0, 0);
    7575        if (result == AS_MAP_FAILED) {
    76                 close(fd);
     76                vfs_put(fd);
    7777                return NULL;
    7878        }
    … …  
    102102
    103103        as_area_destroy(buffer);       
    104         close(fd);
     104        vfs_put(fd);
    105105       
    106106        return NULL;

  • uspace/app/tester/vfs/vfs1.c

    rb19e892 r9c4cf0d  
    108108        }
    109109       
    110         close(fd0);
     110        vfs_put(fd0);
    111111       
    112112        const char *rv = read_root();

  • uspace/app/trace/trace.c

    rb19e892 r9c4cf0d  
    531531        if (fd_root >= 0) {
    532532                rc = loader_add_inbox(ldr, "root", fd_root);
    533                 close(fd_root);
     533                vfs_put(fd_root);
    534534                if (rc != EOK)
    535535                        goto error;
    … …  
    705705        o = oper_new("write", 3, arg_def, V_ERRNO, 1, resp_def);
    706706        proto_add_oper(p, VFS_IN_WRITE, o);
    707         o = oper_new("truncate", 5, arg_def, V_ERRNO, 0, resp_def);
     707        o = oper_new("vfs_resize", 5, arg_def, V_ERRNO, 0, resp_def);
    708708        proto_add_oper(p, VFS_IN_RESIZE, o);
    709         o = oper_new("fstat", 1, arg_def, V_ERRNO, 0, resp_def);
     709        o = oper_new("vfs_stat", 1, arg_def, V_ERRNO, 0, resp_def);
    710710        proto_add_oper(p, VFS_IN_STAT, o);
    711         o = oper_new("close", 1, arg_def, V_ERRNO, 0, resp_def);
    712         proto_add_oper(p, VFS_IN_CLOSE, o);
    713         o = oper_new("mount", 2, arg_def, V_ERRNO, 0, resp_def);
     711        o = oper_new("vfs_put", 1, arg_def, V_ERRNO, 0, resp_def);
     712        proto_add_oper(p, VFS_IN_PUT, o);
     713        o = oper_new("vfs_mount", 2, arg_def, V_ERRNO, 0, resp_def);
    714714        proto_add_oper(p, VFS_IN_MOUNT, o);
    715715/*      o = oper_new("unmount", 0, arg_def);
    716716        proto_add_oper(p, VFS_IN_UNMOUNT, o);*/
    717         o = oper_new("sync", 1, arg_def, V_ERRNO, 0, resp_def);
     717        o = oper_new("vfs_sync", 1, arg_def, V_ERRNO, 0, resp_def);
    718718        proto_add_oper(p, VFS_IN_SYNC, o);
    719719        o = oper_new("rename", 0, arg_def, V_ERRNO, 0, resp_def);
    720720        proto_add_oper(p, VFS_IN_RENAME, o);
    721         o = oper_new("statfs", 0, arg_def, V_ERRNO, 0, resp_def);
     721        o = oper_new("vfs_statfs", 0, arg_def, V_ERRNO, 0, resp_def);
    722722        proto_add_oper(p, VFS_IN_STATFS, o);
    723         o = oper_new("walk", 2, arg_def, V_INT_ERRNO, 0, resp_def);
     723        o = oper_new("vfs_walk", 2, arg_def, V_INT_ERRNO, 0, resp_def);
    724724        proto_add_oper(p, VFS_IN_WALK, o);
    725         o = oper_new("open", 2, arg_def, V_ERRNO, 0, resp_def);
     725        o = oper_new("vfs_open", 2, arg_def, V_ERRNO, 0, resp_def);
    726726        proto_add_oper(p, VFS_IN_OPEN, o);
    727         o = oper_new("unlink", 3, arg_def, V_ERRNO, 0, resp_def);
     727        o = oper_new("vfs_unlink", 3, arg_def, V_ERRNO, 0, resp_def);
    728728        proto_add_oper(p, VFS_IN_UNLINK, o);
    729729

  • uspace/app/viewer/viewer.c

    rb19e892 r9c4cf0d  
    116116        int rc = vfs_stat(fd, &stat);
    117117        if (rc != EOK) {
    118                 close(fd);
     118                vfs_put(fd);
    119119                return false;
    120120        }
    … …  
    122122        void *tga = malloc(stat.size);
    123123        if (tga == NULL) {
    124                 close(fd);
     124                vfs_put(fd);
    125125                return false;
    126126        }
    … …  
    129129        if ((rd < 0) || (rd != (ssize_t) stat.size)) {
    130130                free(tga);
    131                 close(fd);
    132                 return false;
    133         }
    134        
    135         close(fd);
     131                vfs_put(fd);
     132                return false;
     133        }
     134       
     135        vfs_put(fd);
    136136       
    137137        *p_local_surface = decode_tga(tga, stat.size, 0);

  • uspace/app/websrv/websrv.c

    rb19e892 r9c4cf0d  
    245245               
    246246                if (nr < 0) {
    247                         close(fd);
     247                        vfs_put(fd);
    248248                        return EIO;
    249249                }
    … …  
    252252                if (rc != EOK) {
    253253                        fprintf(stderr, "tcp_conn_send() failed\n");
    254                         close(fd);
     254                        vfs_put(fd);
    255255                        return rc;
    256256                }
    257257        }
    258258       
    259         close(fd);
     259        vfs_put(fd);
    260260       
    261261        return EOK;
Note: See TracChangeset for help on using the changeset viewer.