kpartx: use unsigned int for "ns" argument of ptreader
authorMartin Wilck <mwilck@suse.com>
Sat, 12 Oct 2019 21:28:48 +0000 (21:28 +0000)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Mon, 2 Mar 2020 08:19:39 +0000 (09:19 +0100)
ns, the number of slices, should be positive.

Signed-off-by: Martin Wilck <mwilck@suse.com>
kpartx/bsd.c
kpartx/dasd.c
kpartx/dos.c
kpartx/gpt.c
kpartx/gpt.h
kpartx/kpartx.h
kpartx/mac.c
kpartx/ps3.c
kpartx/solaris.c
kpartx/sun.c
kpartx/unixware.c

index f87175e..0e661fb 100644 (file)
@@ -47,13 +47,13 @@ struct bsd_disklabel {
 };
 
 int
-read_bsd_pt(int fd, struct slice all, struct slice *sp, int ns) {
+read_bsd_pt(int fd, struct slice all, struct slice *sp, unsigned int ns) {
        struct bsd_disklabel *l;
        struct bsd_partition *p;
        unsigned int offset = all.start, end;
        int max_partitions;
        char *bp;
-       int n = 0, i, j;
+       unsigned int n = 0, i, j;
 
        bp = getblock(fd, offset+1);    /* 1 sector suffices */
        if (bp == NULL)
index 4f3f744..1486cca 100644 (file)
@@ -66,7 +66,7 @@ typedef unsigned int __attribute__((__may_alias__)) label_ints_t;
  */
 int
 read_dasd_pt(int fd, __attribute__((unused)) struct slice all,
-            struct slice *sp, __attribute__((unused)) int ns)
+            struct slice *sp, __attribute__((unused)) unsigned int ns)
 {
        int retval = -1;
        int blocksize;
index 4985152..0c70669 100644 (file)
@@ -74,10 +74,10 @@ is_gpt(int type) {
 }
 
 int
-read_dos_pt(int fd, struct slice all, struct slice *sp, int ns) {
+read_dos_pt(int fd, struct slice all, struct slice *sp, unsigned int ns) {
        struct partition p;
        unsigned long offset = all.start;
-       int i, n=4;
+       unsigned int i, n=4;
        unsigned char *bp;
        uint64_t  sector_size_mul = get_sector_size(fd)/512;
 
index aa2a440..785b34e 100644 (file)
@@ -602,11 +602,11 @@ fail:
  */
 int
 read_gpt_pt (int fd, __attribute__((unused)) struct slice all,
-            struct slice *sp, int ns)
+            struct slice *sp, unsigned int ns)
 {
        gpt_header *gpt = NULL;
        gpt_entry *ptes = NULL;
-       uint32_t i;
+       unsigned int i;
        int n = 0;
        int last_used_index=-1;
        int sector_size_mul = get_sector_size(fd)/512;
index 7bb54b7..4e1b49a 100644 (file)
@@ -105,7 +105,7 @@ typedef struct _legacy_mbr {
 #define EFI_GPT_PRIMARY_PARTITION_TABLE_LBA 1
 
 /* Functions */
-int read_gpt_pt (int fd, struct slice all, struct slice *sp, int ns);
+int read_gpt_pt (int fd, struct slice all, struct slice *sp, unsigned int ns);
 
 
 #endif
index f228ed8..b20af1d 100644 (file)
@@ -38,7 +38,7 @@ struct slice {
 };
 
 typedef int (ptreader)(int fd, struct slice all, struct slice *sp,
-                      int ns);
+                      unsigned int ns);
 
 extern int force_gpt;
 
@@ -54,7 +54,7 @@ extern ptreader read_ps3_pt;
 
 char *getblock(int fd, unsigned int secnr);
 
-static inline int
+static inline unsigned int
 four2int(unsigned char *p) {
        return p[0] + (p[1]<<8) + (p[2]<<16) + (p[3]<<24);
 }
index 6aeac4e..c21ac70 100644 (file)
@@ -6,12 +6,12 @@
 
 int
 read_mac_pt(int fd, __attribute__((unused)) struct slice all,
-           struct slice *sp, int ns) {
+           struct slice *sp, unsigned int ns) {
        struct mac_driver_desc *md;
        struct mac_partition *part;
        unsigned secsize;
        char *data;
-       int blk, blocks_in_map;
+       unsigned int blk, blocks_in_map;
        int n = 0;
 
        md = (struct mac_driver_desc *) getblock(fd, 0);
index 2cf4ffa..42551bc 100644 (file)
@@ -34,7 +34,7 @@ struct disklabel {
 static int
 read_disklabel(int fd, struct disklabel *label) {
        unsigned char *data;
-       int i;
+       unsigned int i;
 
        for (i = 0; i < sizeof(struct disklabel) / SECTOR_SIZE; i++) {
                data = (unsigned char *) getblock(fd, i);
@@ -49,7 +49,7 @@ read_disklabel(int fd, struct disklabel *label) {
 
 int
 read_ps3_pt(int fd, __attribute__((unused)) struct slice all,
-           struct slice *sp, __attribute__((unused)) int ns) {
+           struct slice *sp, __attribute__((unused)) unsigned int ns) {
        struct disklabel label;
        int n = 0;
        int i;
index e7826c6..c2480b5 100644 (file)
@@ -28,11 +28,11 @@ struct solaris_x86_vtoc {
 };
 
 int
-read_solaris_pt(int fd, struct slice all, struct slice *sp, int ns) {
+read_solaris_pt(int fd, struct slice all, struct slice *sp, unsigned int ns) {
        struct solaris_x86_vtoc *v;
        struct solaris_x86_slice *s;
        unsigned int offset = all.start;
-       int i, n;
+       unsigned int i, n;
        char *bp;
 
        bp = getblock(fd, offset+1);    /* 1 sector suffices */
index 276066d..df630a7 100644 (file)
@@ -59,11 +59,11 @@ sun_verify_checksum (struct sun_disk_label *label)
 }
 
 int
-read_sun_pt(int fd, struct slice all, struct slice *sp, int ns) {
+read_sun_pt(int fd, struct slice all, struct slice *sp, unsigned int ns) {
        struct sun_disk_label *l;
        struct sun_raw_part *s;
        unsigned int offset = all.start, end;
-       int i, j, n;
+       unsigned int i, j, n;
        char *bp;
 
        bp = getblock(fd, offset);
index c7b9786..2f663af 100644 (file)
@@ -48,12 +48,12 @@ struct unixware_disklabel {
 };  /* 408 */
 
 int
-read_unixware_pt(int fd, struct slice all, struct slice *sp, int ns) {
+read_unixware_pt(int fd, struct slice all, struct slice *sp, unsigned int ns) {
        struct unixware_disklabel *l;
        struct unixware_slice *p;
        unsigned int offset = all.start;
        char *bp;
-       int n = 0;
+       unsigned int n = 0;
 
        bp = getblock(fd, offset+29);   /* 1 sector suffices */
        if (bp == NULL)