Question

更新我的Pebble表盘的电池显示器时，我将电池电量提升到int retaattery_level，通过我的toString功能运行它（100％的时间工作，我测试了它），然后追加％和一个空终止符，只是为了衡量。但是，出于某种原因，当该行运行时，当电池从100％下降到90％（或100到任何低位数）时，我会收到堆损坏错误。我尝试删除/添加null终止符，使用strcat（）和strncat（），甚至转义％，似乎没有任何工作。一旦我删除违规行，它就能完美运行。

为什么这条线会破坏堆？可选地，我该如何修复/避免它呢？

static void battery_update_proc(Layer *layer, GContext *ctx) {
  graphics_context_set_text_color(ctx, TxtColor);
  char * lvl = calloc(6,sizeof(char));
  lvl = toString(s_battery_level, lvl);
  //strncat(lvl, "%\0", 2);
  //offending line above ^
  APP_LOG(APP_LOG_LEVEL_DEBUG, "%s", lvl);
  graphics_draw_text(ctx, lvl, s_txt_font, GRect(bound.origin.x + ROUND_OFFSET_BA_X, bound.size.h * .5 + ROUND_OFFSET_BA_Y, bound.size.w, 24), GTextOverflowModeWordWrap, GTextAlignmentCenter, NULL);
}

为了更好的衡量，这里是toString函数：

char * toString(int value, char * result) {
    int digit = digits(value);
    result = calloc(digit + 1, sizeof(char));
    result[digit] = '\0';
    int usedVal = 0;
    for (int i = digit; i > 0; i--)
    {
        int x = (value - usedVal) / pwrOf10(i - 1);
        result[digit - i] = (char) x + '0';
        usedVal = usedVal + (result[digit - i] - '0') * pwrOf10(i - 1);
    }
    return result;
}

int digits(int n) {
    if (n < 0) return digits((n == 0) ? 9999 : -n);
    if (n < 10) return 1;
    return 1 + digits(n / 10);
}

int pwrOf10(int power) {
    int val = 1;
    int i = power;
    while (i > 0)
    {
        val *= 10;
        i--;
    }
return val;
}

Answer 1

您为lvl分配内存两次 - 一次在battery_update_proc，然后又在toString。这是问题的第二次，因为它只为数字分配了足够的空间。

然后你不是free任何一块内存，所以你也有内存泄漏。

至于如何修复它，为什么不将toString的调用替换为sprintf(lvl,"%d%%",s_battery_level)，然后记住在函数末尾调用free(lvl)。

Answer 2

问题的最终答案是我必须修改toString以在calloc（）调用中添加额外的空间。感谢所有提醒我技术性的人！

鹅卵石堆腐败与Str（n）猫

2 个答案: