Project

General

Profile

Actions

Bug #13747

open

udfs: variable may be used uninitialized

Added by Toomas Soome 12 days ago. Updated 12 days ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
filesystems (not ZFS)
Start date:
Due date:
% Done:

90%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

Build errors with shadow gcc 7:

    fsdb.c: In function 'get_blkno':
    fsdb.c:852:9: error: 'lad' may be used uninitialized in this function [-Werror=maybe-uninitialized]
      852 |     len = SWAP_32(lad->lad_ext_len);
    fsdb.c:847:9: error: 'sad' may be used uninitialized in this function [-Werror=maybe-uninitialized]
      847 |     len = SWAP_32(sad->sad_ext_len);
    cc1: all warnings being treated as errors

    fsdb.c:852:16: error: 'len' may be used uninitialized in this function [-Werror=maybe-uninitialized]
        flags = len >> 30;
                ~~~~^~~~~
    fsdb.c:859:17: error: 'bno' may be used uninitialized in this function [-Werror=maybe-uninitialized]
         de[d].blkno = bno;
         ~~~~~~~~~~~~^~~~~
    fsdb.c:857:15: error: 'prn' may be used uninitialized in this function [-Werror=maybe-uninitialized]
         de[d].prn = prn;
         ~~~~~~~~~~^~~~~
    cc1: all warnings being treated as errors
    *** Error code 1
    dmake: Fatal error: Command failed for target `fsdb.o'
    Current working directory /code/illumos-gate/usr/src/cmd/fs.d/udfs/fsdb

Cleaned up all uninitialized warnings in udfs tree.

Testing done: build/install/boot

Actions #1

Updated by Toomas Soome 12 days ago

  • Subject changed from udfs/fsdb: variable may be used uninitialized to udfs: variable may be used uninitialized
  • Description updated (diff)
Actions #2

Updated by Toomas Soome 12 days ago

  • Description updated (diff)
Actions #3

Updated by Electric Monk 12 days ago

  • Gerrit CR set to 1440
Actions

Also available in: Atom PDF