Re: [PATCH] Fix: ArithmeticError: Could not align to closest sector (#561278)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-                f.start = disk.format.alignment.alignNearest(f, f.start)
+                try:
+                    f.start = disk.format.alignment.alignNearest(f, f.start)
+                except ArithmeticError, e:
+                    # This happens when the free region is too small to create
+                    # an aligned partition in it, ie the freespace between the
+                    # mbr and the first aligned partition
+                    continue

              if not disk.format.endAlignment.isAligned(f, f.end):
-                f.end = disk.format.endAlignment.alignNearest(f, f.end)
+                try:
+                    f.end = disk.format.endAlignment.alignNearest(f, f.end)
+                except ArithmeticError, e:
+                    # This happens when the free region is too small to create
+                    # an aligned partition in it, ie the freespace after the
+                    # last aligned partition
+                    continue

              if f.length>  0:
                  free.append(f)


Looks good.
Maybe just one thing: two copies of the same comment. I think a kind reader will understand if the second comment is left out.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux