am 1206e872: resolved conflicts for merge of 09f774b7 to gingerbread-plus-aosp

* commit '1206e872ce74aab253c39c3547bfaadc5e1f6011':
  vold: fix an offset one bug that makes partition 4 unusable
  vold: set state back to idle on formatting error
gugelfrei
Jean-Baptiste Queru 14 years ago committed by Android Git Automerger
commit 2996451f77

@ -197,8 +197,8 @@ void DirectVolume::handlePartitionAdded(const char *devpath, NetlinkEvent *evt)
#ifdef PARTITION_DEBUG #ifdef PARTITION_DEBUG
SLOGD("Dv:partAdd: part_num = %d, minor = %d\n", part_num, minor); SLOGD("Dv:partAdd: part_num = %d, minor = %d\n", part_num, minor);
#endif #endif
if (part_num >= MAX_PARTITIONS) { if (part_num > MAX_PARTITIONS) {
SLOGE("Dv:partAdd: ignoring part_num = %d (max: %d)\n", part_num, MAX_PARTITIONS-1); SLOGE("Dv:partAdd: ignoring part_num = %d (max: %d)\n", part_num, MAX_PARTITIONS);
} else { } else {
mPartMinors[part_num -1] = minor; mPartMinors[part_num -1] = minor;
} }

@ -219,6 +219,7 @@ int Volume::formatVol() {
setState(Volume::State_Formatting); setState(Volume::State_Formatting);
int ret = -1;
// Only initialize the MBR if we are formatting the entire device // Only initialize the MBR if we are formatting the entire device
if (formatEntireDevice) { if (formatEntireDevice) {
sprintf(devicePath, "/dev/block/vold/%d:%d", sprintf(devicePath, "/dev/block/vold/%d:%d",
@ -242,10 +243,11 @@ int Volume::formatVol() {
goto err; goto err;
} }
setState(Volume::State_Idle); ret = 0;
return 0;
err: err:
return -1; setState(Volume::State_Idle);
return ret;
} }
bool Volume::isMountpointMounted(const char *path) { bool Volume::isMountpointMounted(const char *path) {

Loading…
Cancel
Save